chore(region-redirect-middleware): make bucket region header a prerequisite

Author: smilkuri

packages/middleware-sdk-s3/src/region-redirect-middleware.ts

@@ -39,19 +39,20 @@ export function regionRedirectMiddleware(clientConfig: PreviouslyResolved): Init
           const statusCode = err?.$metadata?.httpStatusCode;
           const isHeadBucket = context.commandName === "HeadBucketCommand";
           const bucketRegionHeader = err?.$response?.headers?.["x-amz-bucket-region"];
-          if (
-            statusCode === 301 ||
-            (statusCode === 400 &&
-              (err?.name === "IllegalLocationConstraintException" || (isHeadBucket && bucketRegionHeader)))
-          ) {
-            try {
-              const actualRegion = bucketRegionHeader;
-              context.logger?.debug(`Redirecting from ${await clientConfig.region()} to ${actualRegion}`);
-              context.__s3RegionRedirect = actualRegion;
-            } catch (e) {
-              throw new Error("Region redirect failed: " + e);
+          if (bucketRegionHeader) {
+            if (
+              statusCode === 301 ||
+              (statusCode === 400 && (err?.name === "IllegalLocationConstraintException" || isHeadBucket))
+            ) {
+              try {
+                const actualRegion = bucketRegionHeader;
+                context.logger?.debug(`Redirecting from ${await clientConfig.region()} to ${actualRegion}`);
+                context.__s3RegionRedirect = actualRegion;
+              } catch (e) {
+                throw new Error("Region redirect failed: " + e);
+              }
+              return next(args);
             }
-            return next(args);
           }
         }
         throw err;