fix: added check for existing key in loop

snyk · Sep 21, 2022 · 04c00bc · 04c00bc
1 parent d856544
commit 04c00bc
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/src/lib/ecosystems/resolve-test-facts.ts b/src/lib/ecosystems/resolve-test-facts.ts
@@ -264,12 +264,14 @@ export async function resolveAndTestFactsRegistry(
 
         const vulnerabilities: IssueData[] = [];
         for (const issuesDataKey in response.issuesData) {
-          const issueData = response.issuesData[issuesDataKey];
-          const pkgCoordinate = `${issuesMap[issuesDataKey].pkgName}@${issuesMap[issuesDataKey].pkgVersion}`;
-          issueData.from = [pkgCoordinate];
-          issueData.name = pkgCoordinate;
-          issueData.packageManager = packageManager;
-          vulnerabilities.push(issueData);
+          if (issuesMap[issuesDataKey]) {
+            const issueData = response.issuesData[issuesDataKey];
+            const pkgCoordinate = `${issuesMap[issuesDataKey].pkgName}@${issuesMap[issuesDataKey].pkgVersion}`;
+            issueData.from = [pkgCoordinate];
+            issueData.name = pkgCoordinate;
+            issueData.packageManager = packageManager;
+            vulnerabilities.push(issueData);
+          }
         }
 
         const dependencyCount = response?.depGraphData?.graph?.nodes?.find(