Pin Ruby openssl gem past 3.2.x to clear SBOM EOL finding - #4170
Conversation
|
Co-authored-by: siddharthkp <1863771+siddharthkp@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This pull request updates the demo/lookbook app’s Ruby dependencies to avoid an SBOM false-positive that incorrectly flags the Ruby openssl stdlib gem 3.2.x as the OpenSSL C library 3.2 EOL line.
Changes:
- Add an explicit
openssl (>= 3.3)dependency indemo/Gemfile. - Regenerate
demo/Gemfile.lockso Bundler resolvesopensslto4.0.2and records the new top-level dependency.
Show a summary per file
| File | Description |
|---|---|
| demo/Gemfile | Adds an explicit openssl version constraint to move off 3.2.x. |
| demo/Gemfile.lock | Records the resolved openssl (4.0.2) and adds openssl (>= 3.3) to dependencies. |
Review details
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 1/2 changed files
- Comments generated: 1
- Review effort level: Lite
Authors: Please fill out this form carefully and completely.
Reviewers: By approving this Pull Request you are approving the code change, as well as its deployment and mitigation plans.
Please read this description carefully. If you feel there is anything unclear or missing, please ask for updates.
What are you trying to accomplish?
The
primer-lookbookcontainer is flagged by SBOM-Kustodian EOL ruleeol-eoldate-openssl-008. The match is on the Rubyopensslstdlib gem at3.2.2— its own version line, unrelated to the system OpenSSL C library (3.0.x LTS inruby-base:v3.4-noble, still supported). The scanner just sees the3.2.xstring and maps it to the OpenSSL 3.2 library EOL entry.Changes:
demo/Gemfile— declare the gem explicitly so it is no longer resolved as a default gem:demo/Gemfile.lock— regenerated; addsopenssl (4.0.2). No other gems changed.Integration
No production code changes; demo/lookbook deployable only.
Risk Assessment
Single dependency pin in the demo app; revert is a one-line change.
What approach did you choose and why?
Version-pinning the gem is the cheapest way to make the scanner stop matching, versus filing a suppression/exception per scan cycle. Nothing in the dependency graph caps
openssl, so resolution was clean and did not perturb any other gem.Anything you want to highlight for special attention from reviewers?
Bundler resolved to
openssl 4.0.2rather than a3.3.xrelease —>= 3.3accepts the latest, and the runtime image (Ruby 3.4) supports it. If a3.3.xceiling is preferred for conservatism,"~> 3.3"would pin it there instead.Merge checklist
Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.