Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change duration measurement from days to hours #1046

Closed
wants to merge 1 commit into from
Closed

Change duration measurement from days to hours #1046

wants to merge 1 commit into from

Conversation

jarilaos
Copy link

@jarilaos jarilaos commented Apr 1, 2021

The default value for the lifetime of a legit ticket is 10h, so a longer one could be suspicious.
It is not possible to set a lifetime shorter than 1 day due to the fact that the data type of the duration parameter is int and it is measured in days.

The measurement is changed from days to hours because the value considered opsec is 10h. The default value for the duration parameter is still 10 years for anyone who doesn't care about opsec. Now the data type is float in case someone wants to set a decimal value.

The default value for the lifetime of a legit ticket is 10h, so a longer one could be suspicious.
It is not possible to set a lifetime shorter than 1 day due to the fact that the data type of the duration parameter is int and it is measured in days.

The measurement is changed from days to hours because the value considered opsec is 10h. The default value for the duration parameter is still 10 years for anyone who doesn't care about opsec. The data type is now float in case someone wants to set a decimal value.
@anadrianmanrique
Copy link
Contributor

these changes were merged in the context of #1411.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants