Skip to content

Ignore false positives from osv-scanner #1005

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 9, 2025

Conversation

mhucka
Copy link
Contributor

@mhucka mhucka commented Mar 9, 2025

OSV prior to version 2.0 is unable to parse pip version specs correctly, as explained by one of the developers in the following comment from January 2025:

google/osv-scanner#1483 (comment)

The suggested workaround is to configure osv-scanner to ignore the particular cases it complains about. The following are all about NumPy, because osv-scanner can't understand the version spec "numpy>=1.24,<2.0" and therefore raises errors about all versions of NumPy, including very old ones. Ignoring these specific dependencies is okay because we will never use the old versions of NumPy and it doesn't block detection of future new vulnerabilities.

mhucka added 2 commits March 8, 2025 20:19
OSV prior to version 2.0 is unable to parse pip version specs
correctly, as explained by one of the developers in the following
comment from January 2025:

google/osv-scanner#1483 (comment)

The suggested workaround is to configure osv-scanner to ignore the particular
cases it complains about. The following are all about NumPy, because
osv-scanner can't understand the version spec "numpy>=1.24,<2.0" and
therefore raises errors about all versions of NumPy, including very old ones.
Ignoring these specific dependencies is okay because we will never use the
old versions of NumPy and it doesn't block detection of future new
vulnerabilities.
@mhucka mhucka marked this pull request as ready for review March 9, 2025 04:22
@mhucka mhucka merged commit 7b8aebd into quantumlib:master Mar 9, 2025
19 checks passed
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.

1 participant