internal/sarif: remove originalURIBaseIds

zpavlinovic · gopherbot · commit 0e39fee41378 · 2024-05-08T21:38:05.000Z
We don't need it and it is not actually needed. Sarif validator is fine without it and so is Github Code scanning. Updates golang/go#61347 Change-Id: I1d368422935fddd6b9960917010287ae7bca2683 Reviewed-on: https://go-review.googlesource.com/c/vuln/+/581076 Run-TryBot: Zvonimir Pavlinovic <zpavlinovic@google.com> Auto-Submit: Zvonimir Pavlinovic <zpavlinovic@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Ian Cottrell <iancottrell@google.com> TryBot-Result: Gopher Robot <gobot@golang.org>
diff --git a/internal/sarif/sarif.go b/internal/sarif/sarif.go
@@ -36,9 +36,6 @@ type Run struct {
 	// Results contain govulncheck findings. There should be exactly one
 	// Result per a detected OSV.
 	Results []Result `json:"results,omitempty"`
-
-	// URIBaseIDs encodes the SARIF originalUriBaseIds property
-	URIBaseIDs map[string]ArtifactLocation `json:"originalUriBaseIds,omitempty"`
 }
 
 // Tool captures information about govulncheck analysis that was run.

Original file line number	Diff line number	Diff line change
`@@ -36,9 +36,6 @@ type Run struct {`
`36`	`36`	`// Results contain govulncheck findings. There should be exactly one`
`37`	`37`	`// Result per a detected OSV.`
`38`	`38`	Results []Result `json:"results,omitempty"`
`39`		`-`
`40`		`- // URIBaseIDs encodes the SARIF originalUriBaseIds property`
`41`		- URIBaseIDs map[string]ArtifactLocation `json:"originalUriBaseIds,omitempty"`
`42`	`39`	`}`
`43`	`40`
`44`	`41`	`// Tool captures information about govulncheck analysis that was run.`