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
Add webroot delay #9650
base: master
Are you sure you want to change the base?
Add webroot delay #9650
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. I'm not so sure whether this feature is worth adding.
I don't think the use case discussed in #9643 warrants the change or that Certbot should go out of its way to cater to such setups.
Tomcat might justify things, but it's probably single/double-digit users and there's a relatively straightforward server.xml
workaround. I think the realer problem with Tomcat is the lack of good documentation about how to use it with Certbot.
The only thing in favor of this flag is that it's simple.
I can understand that, use case probably isn't large indeed, I won't feel offended if you'd close this PR. 😛 But as the issue wasn't closed as "won't fix", I thought I'd just throw the PR together anyway 🙂 |
Fixes #9643.
Not sure if the documentation needs to be updated. I don't think this option will be used very often.
Pull Request Checklist
master
section ofcertbot/CHANGELOG.md
to include a description of the change being made.