Skip to content

feat(WD-24755): added cvss4 support for the CVE page#16379

Open
Skazitron wants to merge 2 commits into
mainfrom
WD-24755-dev-add-the-display-to-the-page
Open

feat(WD-24755): added cvss4 support for the CVE page#16379
Skazitron wants to merge 2 commits into
mainfrom
WD-24755-dev-add-the-display-to-the-page

Conversation

@Skazitron
Copy link
Copy Markdown
Contributor

@Skazitron Skazitron commented May 22, 2026

Done

  • Added support for CVSS4 on the frontend.
  • Added dropdown menu to pick between CVSS3 and CVSS4
  • Added additional accordions to show hide tables based on availability

Note

The CVSS source tab (eg. CNA, NIST, Source 3) doesn't exist on the backend and will be ignored in this PR.

QA

  • Open the demo
  • Alternatively, navigate to /security/cves/ and
    • select a CVE from the list
    • add a CVE suffix to the URL like /security/<cve-id>
  • Ensure layout matches the figma doc.
  • Ensure accordion functions as excpected
  • Ensure tables are dynamically rendered based on whether data is available or not.

Issue / Card

https://warthogs.atlassian.net/browse/WD-24755

Screenshots

image

@webteam-app
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov Bot commented May 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.30%. Comparing base (aadab65) to head (8011d9c).
⚠️ Report is 31 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #16379   +/-   ##
=======================================
  Coverage   48.30%   48.30%           
=======================================
  Files          37       37           
  Lines        5912     5912           
=======================================
  Hits         2856     2856           
  Misses       3056     3056           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mattea-turic
Copy link
Copy Markdown

Looks good @Skazitron!

Just a note on the copy (@eliman11), we should be using en-dashes, so "Severity score breakdown — preview...", should be "Severity score breakdown – preview..."

@Skazitron
Copy link
Copy Markdown
Contributor Author

Skazitron commented May 22, 2026

@mattea-turic TYSM for the fast review! Changing dashes to non-ChatGPT dashes 🫡

@@ -0,0 +1,433 @@
<!doctype html>
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This will be removed after PR review.

@eliman11
Copy link
Copy Markdown

Thanks @Skazitron I see we've accounted for tabs in above this table for some CVEs. Would you happen to know a sample CVE page with the tabs to review?

  • This dropdown should use the small base button styling which has the padding around the text in the button
Screenshot 2026-05-22 at 14 59 05
  • Can we make sure the table is still legible on smaller screens? On some widths parts of the text get truncated. The table should behave the same way as the status table above it. Also the "Vector.." text should wrap onto the next line if the text overflows
Screenshot 2026-05-22 at 15 03 17 Screenshot 2026-05-22 at 15 03 01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants