avoid re-instantiating Response when not required

cloudflare · Nov 2, 2024 · 960dc6a · 960dc6a
1 parent 96c331d
commit 960dc6a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 9 deletions.
diff --git a/packages/next-on-pages/templates/_worker.js/handleRequest.ts b/packages/next-on-pages/templates/_worker.js/handleRequest.ts
@@ -114,15 +114,17 @@ async function generateResponse(
 		});
 	}
 
-	const newHeaders = headers.normal;
-	applyHeaders(newHeaders, resp.headers);
-	applyHeaders(newHeaders, headers.important);
+	if (status || resp.status !== 101) {
+		const newHeaders = headers.normal;
+		applyHeaders(newHeaders, resp.headers);
+		applyHeaders(newHeaders, headers.important);
 
-	resp = new Response(resp.body, {
-		...resp,
-		status: status || resp.status,
-		headers: newHeaders,
-	});
+		resp = new Response(resp.body, {
+			...resp,
+			status: status || resp.status,
+			headers: newHeaders,
+		});
+	}
 
 	return resp;
 }
diff --git a/packages/next-on-pages/templates/_worker.js/utils/routing.ts b/packages/next-on-pages/templates/_worker.js/utils/routing.ts
@@ -131,7 +131,7 @@ export async function runOrFetchBuildOutputItem(
 		return new Response('Internal Server Error', { status: 500 });
 	}
 
-	return createMutableResponse(resp);
+	return resp;
 }
 
 /**