-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add support for CVSS v4 #37
Comments
https://www.first.org/cvss/v4-0/ is just an empty placeholder, it says that "Links on the left lead to CVSS version 4.0's specification and related resources." but that does not seem to be true. Presentation says that "Request for Public Comment: October 2022", do we have anyone on CVSS sig? Previously (3.0, 3.1), we had a lot of problems with computation algorithm description, which was rather vague and different implementations and programming languages got to different values. |
@rehakm is on the CVSS SIG and will find out more about specific dates at the upcoming SIG meeting. |
I have attended the recent CVSS SIG meeting and agreed to have us develop a reference implementation of a calculator (so technically updating this library) in Python. |
hey @skontar I noticed you implemented https://github.com/RedHatProductSecurity/cvss-v4-calculator . does that mean you wont work on the python update for it? |
@marco-silva0000 someone will definitely work on the Python library update. I am not sure if that will be me or anyone else on the team. However, the CVSS v4 is still not mature enough, so I would not recommend jumping on it just yet. The reference implementation you pointed out is not necessarily final yet. |
Looks like it's now final: https://www.first.org/cvss/calculator/4.0 |
I am aware of that. Unfortunately, currently it is not our priority. If anyone is willing to port the logic from https://github.com/RedHatProductSecurity/cvss-v4-calculator and integrate it to this library, it would be appreciated. Otherwise it will take a while. Sorry about that. |
@skontar I can do it, but you don't have a Contributor.md so I am unclear as to how you folks want the integration done. Standard fork and PR for an external contributor? |
Also beware that we need to support Python 2.7 until June 2024. |
I am following the current codebase structure with the constants and class files and shoveling the code from the separate CVSS4 project, so the Python 2.7 support should still be there. Hoping to have a first run by end-of-weekend. |
Hey, are there any updates regarding the CVSS4 integration? |
@TitusA7 Unfortunately (or fortunately depending on your perspective) the day job has gotten in the way. I am hoping to continue on this into the new year as time permits, but I can't make it a priority given what I have on my plate. |
I've added a PR for my implementation: #45 I based it on the js implementation: https://github.com/RedHatProductSecurity/cvss-v4-calculator |
I will keep this open until we create another release. |
Huge thanks to @jobiewinserapck for implementation! |
Is it possible to create a release which supports v4.0? |
Yes. Probably this week. We are still looking at some rounding issues in Javascript implementation, but I think this should not impact precise arithmetic in Python implementation. |
CVSS4 was released in 3.0. I am closing this issue. |
CVSS v4 is around the corner according to: https://www.first.org/cvss/v4-0/cvss-v40-presentation.pdf#page=34
This issue tracks all the necessary work to add v4 support to this library.
Jira issue: SECDATA-77
The text was updated successfully, but these errors were encountered: