RiotGames / key-conjurer

Temporary Credential Service
https://technology.riotgames.com/news/key-conjurer-our-policy-least-privilege
Apache License 2.0
167 stars 33 forks source link

TTL is unitless in the display #12

Closed CtrlZvi closed 4 years ago

CtrlZvi commented 5 years ago

Could the TTL display be updated to include the units? I know from experience that the numbers represent hours, but the UI does not make that clear, and coming from a DNS background, my brain keeps wanting it to be seconds.

TheM0ng00se commented 4 years ago

Hey @CtrlZvi , this displays the units now. Please feel free to reopen if you think this is not addressed.

TheM0ng00se commented 4 years ago

TL;DR This is the WebUI for Keyconjurer displaying just an integer without units associated with it.

TheM0ng00se commented 4 years ago

Closing out, @CtrlZvi , please reopen if you have issues.