Skip to content

Conversation

@vladislav-sidorovich
Copy link
Member

@vladislav-sidorovich vladislav-sidorovich commented Nov 21, 2025

We already have some know CVEs in the project. These CVEs must be explicitly mentioned in this file with a justification.

Both Yaml and Json formats can be supported with the single JSONSchema. In this example I use yml because it seems more human redable in case of multiple lines justification.

Examples:

# yaml-language-server: $schema=./known_cves_schema.json

- CVE: CVE-2025-52999
  artifact: org.example:vulnerability-lib:3.18.0
  justification: |
    Some text
    with very nice and clear explanation
  expiration_date: 2030-05-18
[
  {
    "CVE": "CVE-2025-52999",
    "artifact": "org.apache.commons:commons-lang3:3.18.0",
    "justification": "We can and we do",
    "expiration_date": "ISO something"
  },
...
]

@github-actions
Copy link

Code Coverage Report

Overall Project 61.65% 🍏

There is no coverage information present for the Files changed

Copy link
Member

@misolt misolt left a comment

Choose a reason for hiding this comment

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

Looks good, though it would be nice to add integration with an automated tool for detecting vulnerabilities.

I'd like to have the option to run a script that will check the project and show me only the findings that are not already excluded.

@vladislav-sidorovich
Copy link
Member Author

Looks good, though it would be nice to add integration with an automated tool for detecting vulnerabilities.

I'd like to have the option to run a script that will check the project and show me only the findings that are not already excluded.

Yes, exactly it's the next step after defining the schema.

@vladislav-sidorovich vladislav-sidorovich changed the title [b/462688456] Add kwon CVEs files and schema. [b/462688456] Add known CVEs files and schema. Nov 21, 2025
@vladislav-sidorovich vladislav-sidorovich merged commit be33373 into main Nov 21, 2025
11 checks passed
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.

3 participants