Merge pull request #85 from snyk/fix/sarif-format-file-trimming

fix: fixed sarif converter filepath trimming
snyk · May 26, 2021 · eb157b6 · eb157b6
2 parents eaa109b + 0170b61
commit eb157b6
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/sarif_converter.ts b/src/sarif_converter.ts
@@ -48,7 +48,7 @@ function getSuggestions(analysisResults: IAnalysisResult): ISarifSuggestions {
         suggestions[issueId] = [];
       }
       issues.forEach(issue => {
-        suggestions[issueId].push({ ...issue, file: file.substring(1) });
+        suggestions[issueId].push({ ...issue, file: file.charAt(0) === '/' ? file.substring(1) : file });
       });
     }
   }
@@ -172,7 +172,7 @@ function getResults(allIssues: ISarifSuggestion[]): Result[] {
               id: i,
               physicalLocation: {
                 artifactLocation: {
-                  uri: position.file.substring(1),
+                  uri: position.file.charAt(0) === '/' ? position.file.substring(1) : position.file,
                   uriBaseId: '%SRCROOT%',
                 },
                 region: {