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

Making time to wait configurable #744

Merged
merged 3 commits into from Jan 4, 2021

Conversation

danil-smirnov
Copy link
Contributor

Could we have a waiting time configurable? I'd like to set it to half an hour for one of my projects. Are there any reason why it's hardcoded? Thanks

@buchdag buchdag added the kind/feature-request Issue requesting a new feature label Dec 29, 2020
@buchdag
Copy link
Member

buchdag commented Dec 29, 2020

If you could add the relevant documentation entry (here) and pick a variable name a bit more explicit about its purpose (CERTS_REFRESH_INTERVAL or CERTS_CHECK_INTERVAL maybe ?), I'd be happy to merge this.

@danil-smirnov
Copy link
Contributor Author

@buchdag Hi Nicolas! I suppose CERTS_UPDATE_INTERVAL is even better... I've added some docs - feel free to update the text if needed. Thanks

@buchdag
Copy link
Member

buchdag commented Jan 4, 2021

Looks good to me, thanks for the contribution ! 😃

@buchdag buchdag linked an issue Jan 4, 2021 that may be closed by this pull request
@buchdag buchdag merged commit 5d890e7 into nginx-proxy:master Jan 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature-request Issue requesting a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Certs refresh timer
2 participants