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

new_expire_time value is interpreted as integer #132

Closed
peterdragun opened this issue Aug 16, 2021 · 4 comments
Closed

new_expire_time value is interpreted as integer #132

peterdragun opened this issue Aug 16, 2021 · 4 comments

Comments

@peterdragun
Copy link
Contributor

If I use default value for new_expire_time which is "+172800" this is interpreted as number 172800. Problem is that Tower expects either time to be added or timestamp in the future or exact time.
Reproducer:
using latest version of broker with not new_expire_time variable set in config or in config with +.
broker extend --all

@ogajduse
Copy link
Member

If the new_expire_time is passed as a command-line argument, broker interprets it as string, which is correct.

@ogajduse
Copy link
Member

This is a Dynaconf bug, but I would like to keep this one open until the Dynaconf issue is resolved.
dynaconf/dynaconf#585

@JacobCallahan
Copy link
Member

the dynaconf issue has been resolved in the latest release

@ogajduse
Copy link
Member

FYI, I just found out that the latest Dynaconf 3.1.7 does not include the fix yet.

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

No branches or pull requests

3 participants