Closed kevin-paladin closed 1 month ago
The recent updates to the AssetTypeGroupedVulnerabilitiesRule.java
and its corresponding test file involve changes in how vulnerability details are extracted and processed. The code now iterates through a new structure to gather CVE details and reference URLs, creating CveDetails
objects accordingly. The test methods have been updated to reflect these changes, ensuring that the new structure and content are correctly handled.
Files | Change Summary |
---|---|
jobs/pacman-awsrules/src/main/java/.../AssetTypeGroupedVulnerabilitiesRule.java |
Modified to iterate through vulnerabilityItem objects for extracting CVE details and reference URLs, creating CveDetails objects, and populating cveList . |
jobs/pacman-awsrules/src/test/java/.../AssetTypeGroupedVulnerabilitiesRuleTest.java |
Updated correctlyProcessCritical and getExpectedVulnerabilityInfo methods to handle the new format and structure of vulnerability data. |
In code's vast land, a change does bloom,
CVEs now dance in a different room.
Details extracted with careful grace,
Tests aligned to keep the pace.
Vulnerabilities now clear,
In structured form, they do appear.
A rabbit smiles, changes cheer! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Issues
1 New issue
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
This has been superseded
Get the CVE list from 'vulnerabilities' field to allow more than one per finding
Prior to this, the CVE list was always one item.
Type of change
How Has This Been Tested?
The associated tests have been updated and verified.
Checklist:
Summary by CodeRabbit
Bug Fixes
Tests