Adds a priority score to findings#5349
Adds a priority score to findings#5349cat-passwd wants to merge 1 commit intoDependencyTrack:masterfrom
Conversation
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) |
b80229a to
071fd06
Compare
Implements DependencyTrack#5348 Signed-off-by: Antoine FELIX <pro@antoine-felix.fr>
071fd06 to
de78e6d
Compare
|
I work in the medical device manufacturing space so severity and exploitability are considered carefully. My question is on the source of the formula you propose " priorityScore = (1 + epss^0.5) * cvss. CVSSv3 is preferred over CVSSv2 if present." From the design/develop/manufacture/ship perspective CVSSv2 is more expressive and valuable compared to v3. Can you comment on the formula and comment on the v2 and v3 proposal? From my perspective using multiple data sources as you suggest is excellent. My request would be to have a choice of the priorityScore formula for me the user to select or to implement on my own. The medical device standards groups are and have been struggling with the various ways to assess the scoring with many proposed ways to do so, and all of the formulas have issues so far. This is an on going effort in the standards bodies, where I participate by offering real world experience. Thanks. |
|
Combining EPSS and CVSS scores as vulnerability priority is a good idea. At the same time, KEV data should be included (KEV can be simply considered as EPSS = 1.0), and vulnerability priority should be used to calculate metrics instead of CVSS ratings. Here, I propose using ratings instead of scores, which helps to maintain conceptual consistency with CVSS ratings. |
|
Hi @retanoj ! Thanks for your interest on this matter, I agree with your assessment regarding the importance of using the KEV, emulating an EPSS of 1.0 when a vulnerability is included in it:
However, I don't understand your proposition of using ratings instead of scores. In particular, you mention "CVSS ratings" which surprises me since CVSS stands for "Common Vulnerability Scoring System", clearing meant to output a score and not a rating. Could you please clarify on this last point? |
|
Hi @cat-passwd I know that CVSS is a score system and it maps score ranges to different rating. |

Description
Adding a priority score for project findings, following the formula:
priorityScore = (1 + epss^0.5) * cvss.CVSSv3 is preferred over CVSSv2 if present.
EPSS defaults to 0 when undefined.
The priority score itself defaults to zero when neither CVSSv3 nor CVSSv2 is available.
Addressed Issue
Implements #5348
Additional Details
View of the prioritization using this score (includes the respective frontend pull request):

Checklist
This PR fixes a defect, and I have provided tests to verify that the fix is effectiveThis PR introduces changes to the database model, and I have added corresponding update logic