dashboard/app: tolerate gs:// prefixed in gcs urls

tarasmadan · tarasmadan · commit b0b96efc2991 · 2024-12-20T10:29:36.000+01:00
diff --git a/dashboard/app/api.go b/dashboard/app/api.go
@@ -186,7 +186,7 @@ func gcsPayloadHandler(handler APIHandler) APIHandler {
 			return nil, fmt.Errorf("gcs.NewClient: %w", err)
 		}
 		defer clientGCS.Close()
-		gcsFile, err := clientGCS.Read(payloadURL)
+		gcsFile, err := clientGCS.Read(strings.TrimPrefix(payloadURL, "gs://"))
 		if err != nil {
 			return nil, fmt.Errorf("clientGCS.Read: %w", err)
 		}

Original file line number	Diff line number	Diff line change
`@@ -186,7 +186,7 @@ func gcsPayloadHandler(handler APIHandler) APIHandler {`
`186`	`186`	`return nil, fmt.Errorf("gcs.NewClient: %w", err)`
`187`	`187`	`}`
`188`	`188`	`defer clientGCS.Close()`
`189`		`- gcsFile, err := clientGCS.Read(payloadURL)`
	`189`	`+ gcsFile, err := clientGCS.Read(strings.TrimPrefix(payloadURL, "gs://"))`
`190`	`190`	`if err != nil {`
`191`	`191`	`return nil, fmt.Errorf("clientGCS.Read: %w", err)`
`192`	`192`	`}`