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

Add support for Diffie-Hellman (SSL) parameters in VHOST resource. #212

Merged
merged 2 commits into from
Dec 30, 2013

Conversation

hdanes
Copy link
Contributor

@hdanes hdanes commented Dec 27, 2013

Introduced optional parameter for SSL Diffie-Hellman support.

Nginx documentation:
This directive specifies a file containing Diffie-Hellman key agreement protocol cryptographic parameters, in PEM format, utilized for exchanging session keys between server and client.
Syntax: ssl_dhparam file

Example puppet implementation:
ssl => true,
ssl_cert => 'puppet:///modules/modulename/ssl.hostname.crt',
ssl_key => 'puppet:///modules/modulename/ssl.hostname.key',
ssl_dhparam => 'puppet:///modules/modulename/dh.hostname.pem',

@jfryman
Copy link
Contributor

jfryman commented Dec 30, 2013

Looks like I merged in the wrong order, and didn't catch this one. Can you rebase?

@hdanes
Copy link
Contributor Author

hdanes commented Dec 30, 2013

No problem at all, it should be possible to merge this version.

@jfryman
Copy link
Contributor

jfryman commented Dec 30, 2013

Most excellent! Thanks for the contributions!

jfryman pushed a commit that referenced this pull request Dec 30, 2013
Add support for Diffie-Hellman (SSL) parameters in VHOST resource.
@jfryman jfryman merged commit 2b19293 into voxpupuli:master Dec 30, 2013
@hdanes hdanes deleted the ssl_dhparam branch December 30, 2013 17:40
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 this pull request may close these issues.

2 participants