Skip to content

Commit 01f8461

Browse files
authored
fix(express-dev): Disable x-powered-by header (opennextjs#965)
* fix(express-dev): Disable x-powered-by header * changeset
1 parent 31c3740 commit 01f8461

File tree

3 files changed

+15
-4
lines changed

3 files changed

+15
-4
lines changed

.changeset/tame-seas-raise.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(express-dev): Disable x-powered-by header

examples/app-router/app/api/isr/route.ts

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,13 @@ export async function GET(request: NextRequest) {
1515
const manifest = JSON.parse(prerenderManifest);
1616
const previewId = manifest.preview.previewModeId;
1717

18-
const result = await fetch(`https://${request.headers.get("host")}/isr`, {
19-
headers: { "x-prerender-revalidate": previewId },
20-
method: "HEAD",
21-
});
18+
const result = await fetch(
19+
`${request.headers.get("x-forwarded-proto") ?? "https"}://${request.headers.get("host")}/isr`,
20+
{
21+
headers: { "x-prerender-revalidate": previewId },
22+
method: "HEAD",
23+
},
24+
);
2225

2326
return NextResponse.json({
2427
status: 200,

packages/open-next/src/overrides/wrappers/express-dev.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@ import { getMonorepoRelativePath } from "utils/normalize-path";
88

99
const wrapper: WrapperHandler = async (handler, converter) => {
1010
const app = express();
11+
// We disable this cause we wanna use it ourself
12+
// https://stackoverflow.com/a/13055495/16587222
13+
app.disable("x-powered-by");
1114
// To serve static assets
1215
const basePath = NextConfig.basePath ?? "";
1316
app.use(

0 commit comments

Comments
 (0)