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

Support hybrid RSA and ECDSA configuration #1434

Closed
ltning opened this issue Jan 20, 2021 · 2 comments · Fixed by #1446
Closed

Support hybrid RSA and ECDSA configuration #1434

ltning opened this issue Jan 20, 2021 · 2 comments · Fixed by #1446

Comments

@ltning
Copy link
Contributor

ltning commented Jan 20, 2021

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: Any
  • Ruby: Any
  • Distribution: Any
  • Module version: Any

How to reproduce (e.g Puppet code you use)

Attempt to specify more than one ssl_certificate_key and ssl_certificate/chain/fullchain fails.

What are you seeing

Attempting to specify more than one ssl_certificate_key and ssl_certificate/chain/fullchain fails.

What behaviour did you expect instead

These parameters should be Boolean|String|Array rather than the current Boolean|String, and create one ssl_* configuration line for each entry in the array.

Any additional information you'd like to impart

https://nginx.org/en/docs/http/ngx_http_ssl_module.html#ssl_certificate

@PandemiK911
Copy link

Hello,
I've also been limited by this setting : we have to choose between RSA and ECC certificates, despite the Nginx's ability to support both in the same vhost.

Having this supported would be a nice addition

@saz
Copy link
Contributor

saz commented Mar 22, 2021

As I'm also affected by that issue, I'm currently working on a PR.
I hope to get it done today and happy about any feedback

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

Successfully merging a pull request may close this issue.

3 participants