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

key and cert are required under SSL #793

Closed
gerases opened this issue Apr 2, 2016 · 1 comment
Closed

key and cert are required under SSL #793

gerases opened this issue Apr 2, 2016 · 1 comment

Comments

@gerases
Copy link
Contributor

gerases commented Apr 2, 2016

This makes the catalog fail:

if ($ssl == true) {
      if ($ssl_cert == undef) or ($ssl_key == undef) {
          fail('nginx: SSL certificate/key (ssl_cert/ssl_key) and/or SSL Private must be defined and exist on the tar  get system(s)')
      }
    }

But in my case (and it's hard to change due to various reasons), we provide the key and the cert in an include.

Maybe the solution is to define an array param that will say, "hey, don't validate these params because they are defined elsewhere"?

For now, I have to comment out that validation block :(

Another option is to have a param called something like 'ssl_strict_validate' and default it to true?

@3flex
Copy link
Contributor

3flex commented Apr 11, 2016

This is a duplicate of your issue #743

@3flex 3flex closed this as completed Apr 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants