Merge pull request #1737 from akto-api-security/hotfix/adding_null_ch…

…eck_for_references fix: added null check for references and cwes in test report
akto-api-security · Nov 19, 2024 · e1f0260 · e1f0260
2 parents a8247de + 104accb
commit e1f0260
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/...rd/web/polaris_web/web/src/apps/dashboard/pages/testing/vulnerability_report/Category.jsx b/...rd/web/polaris_web/web/src/apps/dashboard/pages/testing/vulnerability_report/Category.jsx
@@ -32,7 +32,7 @@ function Category({ index, issue, subCategoryMap }) {
 
                 {   
                     issue.vulnerableTestingRunResults.map((vulnerableTestingRunResult, index) => {
-                        return ( <Issue testResults={issue.testResults} vulnerableApi={vulnerableTestingRunResult} cwes={issue.cwe} references={issue.references} /> )
+                        return ( <Issue testResults={issue.testResults} vulnerableApi={vulnerableTestingRunResult} cwes={(issue?.cwe || [])} references={(issue?.references || [])} /> )
                     })
                 }
 

diff --git a/...board/web/polaris_web/web/src/apps/dashboard/pages/testing/vulnerability_report/Issue.jsx b/...board/web/polaris_web/web/src/apps/dashboard/pages/testing/vulnerability_report/Issue.jsx
@@ -127,7 +127,7 @@ const Issue = ({ vulnerableApi, references, cwes }) => {
                 <Text variant="headingSm">References</Text>
                 <List type="number">
                     {
-                        references.map((reference) => {
+                        references?.map((reference) => {
                             return (
                                 <List.Item>
                                     <Link target="_blank" url={reference}><Text color="subdued">{reference}</Text></Link>
@@ -142,7 +142,7 @@ const Issue = ({ vulnerableApi, references, cwes }) => {
                 <Text variant="headingSm">CWE</Text>
                 <HorizontalStack gap={2}>
                     {
-                        cwes.map((cweItem) => {
+                        cwes?.map((cweItem) => {
                             return (
                                 <Box>
                                     <Badge size="small">{cweItem}</Badge>
-Original file line number
+Diff line change
@@ Expand Up / @@ -32,7 +32,7 @@ function Category({ index, issue, subCategoryMap }) { @@
                     {
                         issue.vulnerableTestingRunResults.map((vulnerableTestingRunResult, index) => {
-                            return ( <Issue testResults={issue.testResults} vulnerableApi={vulnerableTestingRunResult} cwes={issue.cwe} references={issue.references} /> )
+                            return ( <Issue testResults={issue.testResults} vulnerableApi={vulnerableTestingRunResult} cwes={(issue?.cwe || [])} references={(issue?.references || [])} /> )
                         })
                     }
@@ Expand Down @@