chore: move MaxRedirectsExceededException out of BlobDescriptorStream

BenWhitehead · BenWhitehead · commit 1bf497e3ce21 · 2025-03-14T16:50:08.000-04:00
diff --git a/google-cloud-storage/src/main/java/com/google/cloud/storage/BlobDescriptorStream.java b/google-cloud-storage/src/main/java/com/google/cloud/storage/BlobDescriptorStream.java
@@ -527,12 +527,4 @@ static BlobDescriptorStream create(
     int maxRedirectsAllowed = 3; // TODO: make this configurable in the ultimate public surface
     return new BlobDescriptorStream(state, executor, callable, maxRedirectsAllowed, retryContext);
   }
-
-  static final class MaxRedirectsExceededException extends RuntimeException {
-    private MaxRedirectsExceededException(int maxRedirectAllowed, int actualRedirects) {
-      super(
-          String.format(
-              "max redirects exceeded (max: %d, actual: %d)", maxRedirectAllowed, actualRedirects));
-    }
-  }
 }
diff --git a/google-cloud-storage/src/main/java/com/google/cloud/storage/MaxRedirectsExceededException.java b/google-cloud-storage/src/main/java/com/google/cloud/storage/MaxRedirectsExceededException.java
@@ -0,0 +1,26 @@
+/*
+ * Copyright 2024 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.google.cloud.storage;
+
+final class MaxRedirectsExceededException extends RuntimeException {
+
+  MaxRedirectsExceededException(int maxRedirectAllowed, int actualRedirects) {
+    super(
+        String.format(
+            "max redirects exceeded (actual: %d, max: %d)", actualRedirects, maxRedirectAllowed));
+  }
+}

Original file line number	Diff line number	Diff line change
`@@ -527,12 +527,4 @@ static BlobDescriptorStream create(`
`527`	`527`	`int maxRedirectsAllowed = 3; // TODO: make this configurable in the ultimate public surface`
`528`	`528`	`return new BlobDescriptorStream(state, executor, callable, maxRedirectsAllowed, retryContext);`
`529`	`529`	`}`
`530`		`-`
`531`		`- static final class MaxRedirectsExceededException extends RuntimeException {`
`532`		`- private MaxRedirectsExceededException(int maxRedirectAllowed, int actualRedirects) {`
`533`		`- super(`
`534`		`- String.format(`
`535`		`- "max redirects exceeded (max: %d, actual: %d)", maxRedirectAllowed, actualRedirects));`
`536`		`- }`
`537`		`- }`
`538`	`530`	`}`