secscan: implement basescore/cveid (PROJQUAY-6697) #2684
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This implements FEDRAMP requirements of providing BaseScores and CVE IDs for a given version of software/vulnerabilities associated with it.
The goal is for this to be a temporary stop-gap solution until these values are available through the Clair API and eventually ACS services.
The design of this PR currently utilizes the existing endpoint and builds the two new fields into the
Features
object. This shouldn't have any adverse effect on clients. I've manually tested the UI and everything still looks good/loads as expected.The majority of the changes in this PR come from updating the JSON files for unit-testing, and lots of new line from
make black
linting.Example screenshot of the new JSON response for this endpoint: