Skip to content

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