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 ssl_session_tickets and ssl_session_ticket_key parameters #821

Merged
merged 1 commit into from
Jun 17, 2016
Merged

Add ssl_session_tickets and ssl_session_ticket_key parameters #821

merged 1 commit into from
Jun 17, 2016

Conversation

iain-buclaw-sociomantic
Copy link
Contributor

@@ -206,6 +210,8 @@
$ssl_stapling_responder = undef,
$ssl_stapling_verify = false,
$ssl_session_timeout = '5m',
$ssl_session_tickets = 'on',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should default undef, and be conditional in the template.

Since it was introduced in 1.5.9, and some distros support older versions than that, this directive would be unrecognized by nginx and prevent it from starting.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rebased with suggested default.

@3flex 3flex merged commit 4f91f7c into voxpupuli:master Jun 17, 2016
@iain-buclaw-sociomantic iain-buclaw-sociomantic deleted the ssltickets branch June 17, 2016 14:06
slm0n87 pushed a commit to slm0n87/puppet-nginx that referenced this pull request Mar 7, 2019
Add ssl_session_tickets and ssl_session_ticket_key parameters
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