Skip to content
This repository was archived by the owner on Jan 28, 2025. It is now read-only.

Commit 5b6f848

Browse files
authored
fix(lambda-at-edge): don't rewrite data requests (#719)
1 parent 8e0a79d commit 5b6f848

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

packages/libs/lambda-at-edge/src/default-handler.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -284,16 +284,12 @@ const handleOriginRequest = async ({
284284
let isNonDynamicRoute =
285285
pages.html.nonDynamic[uri] || pages.ssr.nonDynamic[uri] || isPublicFile;
286286

287-
// Handle custom rewrites, but don't rewrite non-dynamic pages or public files per Next.js docs: https://nextjs.org/docs/api-reference/next.config.js/rewrites
288-
if (!isNonDynamicRoute) {
287+
// Handle custom rewrites, but don't rewrite non-dynamic pages, public files or data requests per Next.js docs: https://nextjs.org/docs/api-reference/next.config.js/rewrites
288+
if (!isNonDynamicRoute && !isDataReq) {
289289
const customRewrite = getRewritePath(request.uri, routesManifest);
290290
if (customRewrite) {
291291
request.uri = customRewrite;
292292
uri = normaliseUri(request.uri);
293-
294-
// Set these variables again since URI has changed
295-
isPublicFile = publicFiles[uri];
296-
isDataReq = isDataRequest(uri);
297293
}
298294
}
299295

0 commit comments

Comments
 (0)