Skip to content

Commit 4dc08a9

Browse files
sommeeeerconico974
andauthored
fix: remove internal next headers in middleware response (#777)
* fix: remove internal next headers in middleware response * lint * add space * Create metal-news-watch.md --------- Co-authored-by: conico974 <nicodorseuil@yahoo.fr>
1 parent 9720783 commit 4dc08a9

File tree

2 files changed

+12
-0
lines changed

2 files changed

+12
-0
lines changed

.changeset/metal-news-watch.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@opennextjs/aws": patch
3+
---
4+
5+
fix: remove internal next headers in middleware response

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

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,13 @@ export async function handleMiddleware(
110110
const resHeaders: Record<string, string | string[]> = {};
111111

112112
responseHeaders.delete("x-middleware-override-headers");
113+
/* Next will set the header `x-middleware-set-cookie` when you `set-cookie` in the middleware.
114+
* We can delete it here since it will be set in `set-cookie` aswell. Next removes this header in the response themselves.
115+
* `x-middleware-next` is set when you invoke `NextResponse.next()`. We can delete it here aswell.
116+
*/
117+
responseHeaders.delete("x-middleware-set-cookie");
118+
responseHeaders.delete("x-middleware-next");
119+
113120
const xMiddlewareKey = "x-middleware-request-";
114121
responseHeaders.forEach((value, key) => {
115122
if (key.startsWith(xMiddlewareKey)) {

0 commit comments

Comments
 (0)