check status code

Author: sommeeeer

packages/open-next/src/core/routing/middleware.ts

@@ -26,6 +26,8 @@ const middleMatch = getMiddlewareMatch(
   functionsConfigManifest,
 );
 
+const REDIRECTS = new Set([301, 302, 303, 307, 308]);
+
 type MiddlewareEvent = InternalEvent & {
   responseHeaders?: Record<string, string | string[]>;
   isExternalRewrite?: boolean;
@@ -95,8 +97,10 @@ export async function handleMiddleware(
     url,
     body: convertBodyToReadableStream(internalEvent.method, internalEvent.body),
   } as unknown as Request);
-  // We want to normalize the Location header if it exists. Headers.has() doesn't care about the case of the header name.
-  if (result.headers.has("Location")) {
+
+  const statusCode = result.status;
+  // We want to normalize the Location header if its a redirect response. Headers.has() doesn't care about the case of the header name.
+  if (REDIRECTS.has(statusCode) && result.headers.has("Location")) {
     result.headers.set(
       "Location",
       normalizeLocationHeader(
@@ -105,7 +109,6 @@ export async function handleMiddleware(
       ),
     );
   }
-  const statusCode = result.status;
 
   /* Apply override headers from middleware
     NextResponse.next({