Skip to content

Don't modify SSLContext::DEFAULT_PARAMS - #362

Merged
chrisarcand merged 1 commit into
hashicorp:masterfrom
nevans:stop-modifying-SSL_CONTEXT-DEFAULT_PARAMS
Jan 12, 2026
Merged

Don't modify SSLContext::DEFAULT_PARAMS#362
chrisarcand merged 1 commit into
hashicorp:masterfrom
nevans:stop-modifying-SSL_CONTEXT-DEFAULT_PARAMS

Conversation

@nevans

@nevans nevans commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

The OpenSSL::SSL::SSLContext#set_params method has had secure defaults since ruby 2.4 or 2.5. Since support for ruby versions prior to 3.1 has been dropped, there's no need to modify this anymore.

Also, the openssl 4.0 gem now freezes the DEFAULT_PARAMS hash. So this is needed for compatibility with both openssl 4.0 and ruby 4.0.

The OpenSSL::SSL::SSLContext#set_params has had secure defaults since
ruby 2.4 or 2.5.  Since support for ruby versions prior to 3.1 has been
dropped, there's no need to modify this anymore.

Also, the openssl 4.0 gem now _freezes_ the `DEFAULT_PARAMS` hash.  So
this is needed for compatibility with both openssl 4.0 and ruby 4.0.
@nevans
nevans requested a review from a team as a code owner December 15, 2025 23:34
@nevans

nevans commented Jan 5, 2026

Copy link
Copy Markdown
Contributor Author

@chrisarcand Happy New Year! 😄 Any thoughts on this this PR (and #361, and #360)? This change is needed for compatibility with v4.0.0 of the openssl gem, which is the default version installed with ruby 4.0.

@chrisarcand chrisarcand left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Excellent!

@chrisarcand
chrisarcand merged commit bccaf2a into hashicorp:master Jan 12, 2026
18 of 19 checks passed
@ttstarck

ttstarck commented Feb 2, 2026

Copy link
Copy Markdown

@chrisarcand can a new version of this gem get released that includes this PR?

@chrisarcand

Copy link
Copy Markdown
Member

You bet, sorry for the delay.

@ttstarck

ttstarck commented Feb 9, 2026

Copy link
Copy Markdown

@chrisarcand much appreciated! Thank you for the release!

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.

3 participants