Skip to content

Add "double reported" advisory check#1089

Open
jasnow wants to merge 1 commit into
rubysec:masterfrom
jasnow:extract-pr585-code
Open

Add "double reported" advisory check#1089
jasnow wants to merge 1 commit into
rubysec:masterfrom
jasnow:extract-pr585-code

Conversation

@jasnow
Copy link
Copy Markdown
Member

@jasnow jasnow commented Jun 1, 2026

Add "double reported" advisory check

Copy link
Copy Markdown

@StantonMatt StantonMatt left a comment

Choose a reason for hiding this comment

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

I took a local verification pass on this because it is labeled ready for review and the duplicate-advisory check affects database correctness.

The data change matches the new check: on current origin/master, the touched YARD advisories contain duplicate GHSA 3jfp-46x4-xgfj; on this branch, only gems/yard/CVE-2026-41493.yml remains and the duplicate GHSA entry is gone.

I ran the repo checks locally with Ruby 3.3.11 and project-local gems:

bundle exec rspec spec/advisories_spec.rb
bundle exec rake lint
git diff --check origin/master...HEAD

spec/advisories_spec.rb passed with 58,703 examples. rake lint also passed: schema validation had 1,281 examples and the advisory lint pass had 58,703 examples, all with 0 failures. The visible GitHub checks, GitHub Actions audit and Linter, are green as well.

Looks good from this pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants