-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
nginx: on encountering lua directives, produce a better warning #9475
Conversation
Still need to take a closer look, but given how sticky this problem has been in the past I like the general approach. I'm not overly familiar with lua, but looking at the documentation seems to suggest there are a few other options that might involve lua code; probably theoretically |
certbot-nginx/tests/testdata/etc_nginx/unsupported_directives.conf
Outdated
Show resolved
Hide resolved
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.
This looks great, helpful solution to get a better error message out there, thanks!
Haven't merged it because I wasn't sure if you wanted to add a changelog entry. |
Thank you! |
…ng (certbot#9475)" This reverts commit c178fa8.
nginx: fix performance regression caused by #9475
See #9066.
The code is a little bit layer violating but it seems a bit tricky to do this from
configurator.py
, given that we are swallowing the parsing errors.The output becomes is as below. It's unfortunate that the warning is printed twice, once on presenting the challenge response, and one on cleaning it up. But: this already happens on
master
, the situation is fairly rare and still achieves a big improvement with telling the user what's wrong.The difference compared to
master
is: