Only write to the rawBucket if the value exists. (ossf#2451)

Signed-off-by: Caleb Brown <[email protected]> Signed-off-by: Caleb Brown <[email protected]> Signed-off-by: nathaniel.wert <[email protected]>
N8BWert · Nov 28, 2022 · 2f1c194 · 2f1c194
1 parent c0cf001
commit 2f1c194
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/cron/internal/controller/main.go b/cron/internal/controller/main.go
@@ -168,14 +168,16 @@ func main() {
 		panic(fmt.Errorf("error writing to BlobStore: %w", err))
 	}
 
-	// Raw data.
-	*metadata.ShardLoc = rawBucket + "/" + data.GetBlobFilename("", t)
-	metadataJSON, err = protojson.Marshal(&metadata)
-	if err != nil {
-		panic(fmt.Errorf("error during protojson.Marshal raw: %w", err))
-	}
-	err = data.WriteToBlobStore(ctx, rawBucket, data.GetShardMetadataFilename(t), metadataJSON)
-	if err != nil {
-		panic(fmt.Errorf("error writing to BlobStore raw: %w", err))
+	if rawBucket != "" {
+		// Raw data.
+		*metadata.ShardLoc = rawBucket + "/" + data.GetBlobFilename("", t)
+		metadataJSON, err = protojson.Marshal(&metadata)
+		if err != nil {
+			panic(fmt.Errorf("error during protojson.Marshal raw: %w", err))
+		}
+		err = data.WriteToBlobStore(ctx, rawBucket, data.GetShardMetadataFilename(t), metadataJSON)
+		if err != nil {
+			panic(fmt.Errorf("error writing to BlobStore raw: %w", err))
+		}
 	}
 }