fix: prevent NPE in RemoteStorageHelper.cleanBuckets

The method getEventBasedHold and getTemporaryHold return Boolean objects which are either true, false or null. In case of a null value, this produced a NPE.
googleapis · Aug 31, 2020 · ef31035 · ef31035
1 parent 913b756
commit ef31035
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/google-cloud-storage/src/main/java/com/google/cloud/storage/testing/RemoteStorageHelper.java b/google-cloud-storage/src/main/java/com/google/cloud/storage/testing/RemoteStorageHelper.java
@@ -88,7 +88,8 @@ public void run() {
                                   Storage.BlobField.EVENT_BASED_HOLD,
                                   Storage.BlobField.TEMPORARY_HOLD))
                           .iterateAll()) {
-                    if (blob.getEventBasedHold() == true || blob.getTemporaryHold() == true) {
+                    if (Boolean.TRUE.equals(blob.getEventBasedHold())
+                        || Boolean.TRUE.equals(blob.getTemporaryHold())) {
                       storage.update(
                           blob.toBuilder().setTemporaryHold(false).setEventBasedHold(false).build(),
                           Storage.BlobTargetOption.userProject(