Skip to content

osv: exclude advisories from non applicable ecosystems#1958

Open
crozzy wants to merge 1 commit into
quay:mainfrom
crozzy:exclude-non-applicable-osv-advisories
Open

osv: exclude advisories from non applicable ecosystems#1958
crozzy wants to merge 1 commit into
quay:mainfrom
crozzy:exclude-non-applicable-osv-advisories

Conversation

@crozzy

@crozzy crozzy commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

OSV will include advisories in files that cover more ecosystems than the one requested. This could lead to incorrect parsing logic being used to parse advisories resulting in false-positives. This patch avoids parsing advisories that don't match the requested ecosystem.

@crozzy
crozzy requested review from a team as code owners July 20, 2026 23:04
@crozzy
crozzy requested a review from hdonnay July 20, 2026 23:04
@crozzy

crozzy commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

Before: 717,975 vulns, 613MB
After: 707,034 vulns, 598MB

Better than nothing

OSV will include advisories in files that cover more ecosystems than the
one requested. This could lead to incorrect parsing logic being used to
parse advisories resulting in false-positives. This patch avoids parsing
advisories that don't match the requested ecosystem.

Signed-off-by: crozzy <joseph.crosland@gmail.com>
@crozzy
crozzy force-pushed the exclude-non-applicable-osv-advisories branch from ea33089 to 3f07716 Compare July 20, 2026 23:06
@crozzy
crozzy requested a review from dcaravel July 20, 2026 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant