Skip to content

fix: correct parsing for CVSSv4 strings with Provider Urgency#8377

Merged
jeremylong merged 3 commits intodependency-check:mainfrom
chadlwilson:fix-provider-urgency-type-parsing-from-vector-strings
Apr 2, 2026
Merged

fix: correct parsing for CVSSv4 strings with Provider Urgency#8377
jeremylong merged 3 commits intodependency-check:mainfrom
chadlwilson:fix-provider-urgency-type-parsing-from-vector-strings

Conversation

@chadlwilson
Copy link
Copy Markdown
Collaborator

@chadlwilson chadlwilson commented Mar 19, 2026

Description of Change

Bumps open-vulnerability-clients to 9.0.4 to address underlying issue, confirmed with a fixed test.

Related issues

Have test cases been added to cover the new functionality?

yes

@boring-cyborg boring-cyborg bot added core changes to core tests test cases labels Mar 19, 2026
Signed-off-by: Chad Wilson <29788154+chadlwilson@users.noreply.github.com>
@chadlwilson chadlwilson force-pushed the fix-provider-urgency-type-parsing-from-vector-strings branch from c40b0de to c3e7663 Compare March 19, 2026 16:40
@chadlwilson chadlwilson self-assigned this Mar 19, 2026
@chadlwilson chadlwilson marked this pull request as ready for review April 1, 2026 11:25
@chadlwilson chadlwilson requested a review from Copilot April 1, 2026 11:27
@chadlwilson chadlwilson removed their assignment Apr 1, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Dependency-Check to use a newer open-vulnerability-clients release that fixes CVSSv4 parsing when the vector string includes a non-default Provider Urgency (U:Clear|Green|Amber|Red), and adjusts the CVSSv4 unit test to cover this scenario (fixes #8376).

Changes:

  • Bump io.github.jeremylong:open-vulnerability-clients from 9.0.3 to 9.0.4.
  • Update CvssUtilTest#testVectorToCvssV4 to use a CVSSv4 vector containing supplemental metrics including U:Amber.
  • Add assertions validating supplemental CVSSv4 fields (Safety/Automatable/Recovery/ValueDensity/ResponseEffort/ProviderUrgency) and that threat/environmental scores remain unset.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
pom.xml Updates managed dependency version for open-vulnerability-clients to pick up the CVSSv4 parsing fix.
core/src/test/java/org/owasp/dependencycheck/utils/CvssUtilTest.java Extends CVSSv4 parsing test coverage to include Provider Urgency (U:Amber) and related supplemental fields.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jeremylong jeremylong merged commit 91c6972 into dependency-check:main Apr 2, 2026
21 of 22 checks passed
@chadlwilson chadlwilson deleted the fix-provider-urgency-type-parsing-from-vector-strings branch April 2, 2026 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core changes to core tests test cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OSSIndexAnalyzer cannot parse CVSSv4 data from vector string with non-default Provider Urgency Type

4 participants