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

www/nginx: The GUI parameter "Verify Certificate" does not do anything #1986

Closed
8191 opened this issue Aug 18, 2020 · 5 comments
Closed

www/nginx: The GUI parameter "Verify Certificate" does not do anything #1986

8191 opened this issue Aug 18, 2020 · 5 comments
Assignees

Comments

@8191
Copy link
Member

8191 commented Aug 18, 2020

Describe the bug
The GUI parameter "TLS: Verify Certificate" of an Upstream does not do anything; Nginx always successfully connects to the upstream, regardless of the checkbox.

To Reproduce
Preconditions:
A working Nginx config which acts as a proxy for a page which is upstreamed via HTTPs and an invalid certificate.

Steps to reproduce the behavior:

  1. Go to Upstream > Upstream > Edit
  2. Check 'TLS: Verify Certificate'
  3. Connect via HTTP to the configured Nginx instance
  4. The proxied page is successfully displayed even the certificate could not be verified.

Expected behavior
Nginx shall not connect to the upstream and display an error to the end-user.

@kulikov-a
Copy link
Member

agree. location.conf template does not contain proxy_ssl_verify directive at all.
IMHO location.conf template not updated very long time
SNI directives also not works as expected

@8191
Copy link
Member Author

8191 commented Sep 28, 2020

@AdSchellevis Any chance to pull #1987 for the next release?

@AdSchellevis
Copy link
Member

@8191 since @fabianfrz is the maintainer, we should ask him if we can pull it in. to me it looks like a good addition, although the model change needs a version upgrade and might be backwards incompatible (removing the url attribute).

@fabianfrz fabianfrz self-assigned this Sep 28, 2020
@fabianfrz
Copy link
Member

@AdSchellevis I did the review and yes, I think the model change is needed all the other things are ok beside the whitespace (@8191 probably used some windows editor which is not set up for spaces)

@8191 8191 closed this as completed Dec 18, 2020
@8191
Copy link
Member Author

8191 commented Dec 18, 2020

Fixed by merging #1987

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants