Skip to content
Snippets Groups Projects
Commit 9f062ddf authored by Martin Lowe's avatar Martin Lowe :flag_ca:
Browse files

Merge branch 'zacharysabourin/main/null-cveId-fix' into 'main'

fix: Add missing cveID null check for cve fetching

See merge request !35
parents 75aa65e8 e8f42bce
No related branches found
No related tags found
1 merge request!35fix: Add missing cveID null check for cve fetching
Pipeline #31356 passed
...@@ -175,14 +175,16 @@ public class InternalAdvisoriesPrecacheProvider implements LoadingCacheProvider< ...@@ -175,14 +175,16 @@ public class InternalAdvisoriesPrecacheProvider implements LoadingCacheProvider<
*/ */
private CveData augmentCveData(CveData orig) { private CveData augmentCveData(CveData orig) {
try { try {
// Fetch data from each source for augmenting if (StringUtils.isNotBlank(orig.getId())) {
Optional<CveProjectData> projectData = fetchCveProjectDetails(orig.getId()); // Fetch data from each source for augmenting
Optional<GithubAdvisoriesData> ghAdvisory = fetchGhAdvisoriesDetails(orig.getId()); Optional<CveProjectData> projectData = fetchCveProjectDetails(orig.getId());
Optional<GithubAdvisoriesData> ghAdvisory = fetchGhAdvisoriesDetails(orig.getId());
return orig.toBuilder()
.setSummary(getCveSummary(projectData, ghAdvisory, orig.getId())) return orig.toBuilder()
.setCvss(getImpactScore(projectData, ghAdvisory)) .setSummary(getCveSummary(projectData, ghAdvisory, orig.getId()))
.build(); .setCvss(getImpactScore(projectData, ghAdvisory))
.build();
}
} catch (Exception e) { } catch (Exception e) {
LOGGER.error("Error retrieving CVE detailed data:", e); LOGGER.error("Error retrieving CVE detailed data:", e);
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment