Skip to content

Bump minimum jruby-openssl to 0.15.4 for CVE-2025-46551#1022

Open
hsbt wants to merge 1 commit intomasterfrom
bumpup-jruby-openssl
Open

Bump minimum jruby-openssl to 0.15.4 for CVE-2025-46551#1022
hsbt wants to merge 1 commit intomasterfrom
bumpup-jruby-openssl

Conversation

@hsbt
Copy link
Copy Markdown
Member

@hsbt hsbt commented Mar 31, 2026

I recieved the dependabot alert for GHSA-72qj-48g4-5xgx at ruby/ruby repository. We should bump up to jruby-openssl dependency with fixed version.

/cc @headius

jruby-openssl versions prior to 0.15.4 have hostname verification
disabled by default, allowing man-in-the-middle attacks with valid
certificates for different domains (GHSA-72qj-48g4-5xgx).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 31, 2026 02:11
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the JRuby-specific dependency constraint in openssl.gemspec to ensure JRuby users pick up a jruby-openssl version that includes the fix for CVE-2025-46551 (GHSA-72qj-48g4-5xgx).

Changes:

  • Bump jruby-openssl runtime dependency from ~> 0.14 to the 0.15 series.
  • Enforce a minimum jruby-openssl version of 0.15.4 on the Java/JRuby platform path.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

spec.platform = "java"
spec.files = []
spec.add_runtime_dependency('jruby-openssl', '~> 0.14')
spec.add_runtime_dependency('jruby-openssl', '~> 0.15', '>= 0.15.4')
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.

Suggested change
spec.add_runtime_dependency('jruby-openssl', '~> 0.15', '>= 0.15.4')
spec.add_runtime_dependency('jruby-openssl')

0.15.4 satisfies ~> 0.14, so it appears to be a false positive caused by Dependabot's loose pattern matching.

I wonder if we can avoid it and similar issues in the future by simply dropping the version constraint, since it doesn't seem to be useful anyway.

Copy link
Copy Markdown
Member Author

@hsbt hsbt Mar 31, 2026

Choose a reason for hiding this comment

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

I agreed to remove version constraint too.

@headius Do you have any opinion about that?

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