Skip to content

Fix Oj GHSA-2cw7 patched version#8162

Open
massif-01 wants to merge 1 commit into
github:massif-01/advisory-improvement-8162from
massif-01:codex/fix-oj-ghsa-2cw7
Open

Fix Oj GHSA-2cw7 patched version#8162
massif-01 wants to merge 1 commit into
github:massif-01/advisory-improvement-8162from
massif-01:codex/fix-oj-ghsa-2cw7

Conversation

@massif-01

Copy link
Copy Markdown

This PR corrects the structured fixed version for GHSA-2cw7-v8ff-p88r.

The source repository advisory for the oj RubyGem lists affected versions as < 3.17.2 and patched version 3.17.2.

The global advisory already records last_known_affected_version_range: < 3.17.2, but the structured fixed version is currently 3.17.3. This PR changes it to 3.17.2.

No vulnerability details are changed; this only corrects affected-version metadata.

Validation:

  • Compared the global advisory JSON against the source repository advisory.
  • Ran jq empty on the modified advisory JSON.
  • Ran git diff --check.

@github-actions github-actions Bot changed the base branch from main to massif-01/advisory-improvement-8162 June 27, 2026 07:56
@massif-01 massif-01 marked this pull request as ready for review June 27, 2026 08:03
Copilot AI review requested due to automatic review settings June 27, 2026 08:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects the structured “fixed” version metadata for the Oj RubyGem advisory GHSA-2cw7-v8ff-p88r to match the already-recorded affected range (< 3.17.2) and the upstream advisory’s patched version.

Changes:

  • Updates the affected.ranges[].events[].fixed version from 3.17.3 to 3.17.2 for oj.
  • Keeps last_known_affected_version_range: "< 3.17.2" consistent with the structured range.

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

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