Correctly set scheme for URIs in the SARIF report output

detekt · Jul 30, 2023 · 32961f7 · 32961f7
1 parent a135984
commit 32961f7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/detekt-report-sarif/src/main/kotlin/io/github/detekt/report/sarif/Results.kt b/detekt-report-sarif/src/main/kotlin/io/github/detekt/report/sarif/Results.kt
@@ -58,7 +58,7 @@ private fun Location.toLocation(code: String?): io.github.detekt.sarif4k.Locatio
                     uriBaseID = SRCROOT
                 )
             } else {
-                ArtifactLocation(uri = filePath.absolutePath.invariantSeparatorsPathString)
+                ArtifactLocation(uri = "file:///${filePath.absolutePath.invariantSeparatorsPathString}")
             }
         )
     )

diff --git a/detekt-report-sarif/src/main/kotlin/io/github/detekt/report/sarif/SarifOutputReport.kt b/detekt-report-sarif/src/main/kotlin/io/github/detekt/report/sarif/SarifOutputReport.kt
@@ -63,7 +63,9 @@ class SarifOutputReport : OutputReport() {
                             version = version
                         )
                     ),
-                    originalURIBaseIDS = basePath?.let { mapOf(SRCROOT to ArtifactLocation(uri = "file://$basePath")) },
+                    originalURIBaseIDS = basePath?.let {
+                        mapOf(SRCROOT to ArtifactLocation(uri = "file:///$basePath"))
+                    },
                     results = toResults(detektion)
                 )
             )