diff --git a/google-cloud-storage/src/main/java/com/google/cloud/storage/GrpcStorageOptions.java b/google-cloud-storage/src/main/java/com/google/cloud/storage/GrpcStorageOptions.java index dcadfb401..841343d31 100644 --- a/google-cloud-storage/src/main/java/com/google/cloud/storage/GrpcStorageOptions.java +++ b/google-cloud-storage/src/main/java/com/google/cloud/storage/GrpcStorageOptions.java @@ -767,6 +767,7 @@ protected StorageSettings.Builder setInternalHeaderProvider( private static final class NoopGrpcInterceptorProvider implements GrpcInterceptorProvider, Serializable { + private static long serialVersionUID = -8523033236999805349L; private static final NoopGrpcInterceptorProvider INSTANCE = new NoopGrpcInterceptorProvider(); @Override diff --git a/google-cloud-storage/src/main/java/com/google/cloud/storage/HttpStorageOptions.java b/google-cloud-storage/src/main/java/com/google/cloud/storage/HttpStorageOptions.java index 9d4873e2b..fd2a57311 100644 --- a/google-cloud-storage/src/main/java/com/google/cloud/storage/HttpStorageOptions.java +++ b/google-cloud-storage/src/main/java/com/google/cloud/storage/HttpStorageOptions.java @@ -36,6 +36,8 @@ import com.google.cloud.storage.spi.v1.StorageRpc; import com.google.common.base.MoreObjects; import com.google.common.collect.ImmutableSet; +import java.io.IOException; +import java.io.ObjectInputStream; import java.io.Serializable; import java.util.Set; @@ -52,7 +54,7 @@ public class HttpStorageOptions extends StorageOptions { private static final String DEFAULT_HOST = "https://storage.googleapis.com"; private final HttpRetryAlgorithmManager retryAlgorithmManager; - private final RetryDependenciesAdapter retryDepsAdapter; + private transient RetryDependenciesAdapter retryDepsAdapter; private HttpStorageOptions(Builder builder, StorageDefaults serviceDefaults) { super(builder, serviceDefaults); @@ -93,6 +95,11 @@ public boolean equals(Object obj) { return obj instanceof HttpStorageOptions && baseEquals((HttpStorageOptions) obj); } + private void readObject(ObjectInputStream in) throws IOException, ClassNotFoundException { + in.defaultReadObject(); + this.retryDepsAdapter = new RetryDependenciesAdapter(); + } + public static HttpStorageOptions.Builder newBuilder() { return new HttpStorageOptions.Builder().setHost(DEFAULT_HOST); } @@ -345,8 +352,7 @@ public ServiceRpc create(StorageOptions options) { * We don't yet want to make HttpStorageOptions itself implement {@link RetryingDependencies} but * we do need use it in a couple places, for those we create this adapter. */ - private final class RetryDependenciesAdapter implements RetryingDependencies, Serializable { - + private final class RetryDependenciesAdapter implements RetryingDependencies { private RetryDependenciesAdapter() {} @Override diff --git a/google-cloud-storage/src/main/java/com/google/cloud/storage/Notification.java b/google-cloud-storage/src/main/java/com/google/cloud/storage/Notification.java index f3e2bb81a..f3e0738d5 100644 --- a/google-cloud-storage/src/main/java/com/google/cloud/storage/Notification.java +++ b/google-cloud-storage/src/main/java/com/google/cloud/storage/Notification.java @@ -26,6 +26,7 @@ * details. */ public class Notification extends NotificationInfo { + private static final long serialVersionUID = 3150928330690874200L; private final StorageOptions options; private transient Storage storage; diff --git a/google-cloud-storage/src/main/java/com/google/cloud/storage/StorageOptions.java b/google-cloud-storage/src/main/java/com/google/cloud/storage/StorageOptions.java index 74a6ffb3d..53ad6142e 100644 --- a/google-cloud-storage/src/main/java/com/google/cloud/storage/StorageOptions.java +++ b/google-cloud-storage/src/main/java/com/google/cloud/storage/StorageOptions.java @@ -63,6 +63,8 @@ public abstract class StorageOptions extends ServiceOptions