Skip to content
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

allow to ignore vulnerabilities by sonatype id or cve id #5

Merged
merged 1 commit into from
Mar 20, 2021

Conversation

sseide
Copy link
Contributor

@sseide sseide commented Mar 16, 2021

this patch improves the ignore-id parameter to allow setting CVE ids too in addition to the Sonatype ID.
Furthermore the documentation is updated that "id" means the Sonatype id of a vulnerability.

As a lot of other scanners accept CVE ids for whitelisting (even other scanner using Sonatype OSSIndex for other languages) this functionality was added here too. The old implementation was misleading as it was not documented that the id to ignore is not the wildly used and everywhere (throughout the internet) documented CVE id but the proprietary Sonatype one. The default output of ossaudit was not showing the Sonatype ID too, only the CVE (within the title field - if available)

Thanks,
S. Seide

@illikainen illikainen merged commit b24255d into illikainen:master Mar 20, 2021
@illikainen
Copy link
Owner

Awesome! You're right that it makes sense to ignore by CVE in addition to the SonaType identifiers.

Thanks!

@sseide
Copy link
Contributor Author

sseide commented Mar 24, 2021

thanks - do you mind to create a new release with all the latest updates?

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.

None yet

2 participants