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

fix: prevent NPE in bundler audit #6462

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Conversation

kmort89
Copy link
Contributor

@kmort89 kmort89 commented Feb 12, 2024

Fixes Issue

#6304

Description of Change

Use "2.0" as default CvssV2Data version instead of null.

Have test cases been added to cover the new functionality?

no

@boring-cyborg boring-cyborg bot added the core changes to core label Feb 12, 2024
@kmort89 kmort89 changed the title Fix https://github.com/jeremylong/DependencyCheck/issues/6304 dix https://github.com/jeremylong/DependencyCheck/issues/6304 Feb 12, 2024
@kmort89 kmort89 changed the title dix https://github.com/jeremylong/DependencyCheck/issues/6304 fix: issue https://github.com/jeremylong/DependencyCheck/issues/6304 Feb 12, 2024
@jeremylong jeremylong changed the title fix: issue https://github.com/jeremylong/DependencyCheck/issues/6304 fix: prevent NPE in bundler audit Feb 13, 2024
@jeremylong jeremylong added this to the 9.0.9 milestone Feb 13, 2024
@jeremylong jeremylong merged commit 5df22e2 into jeremylong:main Feb 13, 2024
8 of 9 checks passed
@jeremylong
Copy link
Owner

Thanks for the PR! really appreciate it.

@chadlwilson
Copy link
Contributor

I'm inferring the milestone is wrong here, as ODC 9.0.9 was released Jan 17? Should this be 9.0.10?

@jeremylong jeremylong modified the milestones: 9.0.9, 9.0.10 Feb 26, 2024
@jeremylong
Copy link
Owner

Thank you for catching this.

@chadlwilson
Copy link
Contributor

No worries, was just wondering why the “fix” wasn’t working 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core changes to core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants