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

draft of update for cvss v4 #528

Merged
merged 3 commits into from
Mar 5, 2024
Merged

Conversation

j---
Copy link
Collaborator

@j--- j--- commented Mar 5, 2024

Based on discussion in #284 , I believe this should close #284 if merged.

@ahouseholder
Copy link
Contributor

ahouseholder commented Mar 5, 2024

I made a few formatting changes:

  • add mapping tables instead of logic statements
  • add CVSSv3 subheading to differentiate it from CVSSv4 content
  • remove CVSSv4/v3 inset box that is saying "we'll do v4 later", since we're doing this one now.
  • break up list-oriented paragraphs into lists

Screenshots of the current rendering below:

Screenshot 2024-03-05 at 1 35 06 PM Screenshot 2024-03-05 at 1 37 33 PM

@j---
Copy link
Collaborator Author

j--- commented Mar 5, 2024

I made a few formatting changes:

Thanks, these are great.

Copy link
Contributor

@ahouseholder ahouseholder left a comment

Choose a reason for hiding this comment

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

These are ready to go.

@ahouseholder ahouseholder merged commit d093301 into CERTCC:main Mar 5, 2024
2 checks passed
@ahouseholder ahouseholder modified the milestones: 2Q24, 1Q24 Jan 13, 2025
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.

Add text about how Technical Impact relates to CVSS v4
2 participants