fix: update kms key handling when opening a resumable upload to clear the value in the json to be null rather than empty string (#2939)

BenWhitehead · web-flow · commit 43553dedce33 · 2025-02-18T18:56:46.000-05:00
Related to b/395940239
diff --git a/google-cloud-storage/src/main/java/com/google/cloud/storage/spi/v1/HttpStorageRpc.java b/google-cloud-storage/src/main/java/com/google/cloud/storage/spi/v1/HttpStorageRpc.java
@@ -1082,7 +1082,7 @@ public String open(StorageObject object, Map<Option, ?> options) {
     try {
       String kmsKeyName = object.getKmsKeyName();
       if (kmsKeyName != null && kmsKeyName.contains("cryptoKeyVersions")) {
-        object.setKmsKeyName("");
+        object.setKmsKeyName(Data.nullOf(String.class));
       }
       Insert req =
           storage

Original file line number	Diff line number	Diff line change
`@@ -1082,7 +1082,7 @@ public String open(StorageObject object, Map<Option, ?> options) {`
`1082`	`1082`	`try {`
`1083`	`1083`	`String kmsKeyName = object.getKmsKeyName();`
`1084`	`1084`	`if (kmsKeyName != null && kmsKeyName.contains("cryptoKeyVersions")) {`
`1085`		`- object.setKmsKeyName("");`
	`1085`	`+ object.setKmsKeyName(Data.nullOf(String.class));`
`1086`	`1086`	`}`
`1087`	`1087`	`Insert req =`
`1088`	`1088`	`storage`