fix(api-server): Preserve original host header for proxied API reques…

…ts (#10082) Co-authored-by: Daniel Choudhury <[email protected]>
redwoodjs · Mar 7, 2024 · 5c8d77f · 5c8d77f
1 parent 67df472
commit 5c8d77f
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,8 @@
 # CHANGELOG
 
 ## Unreleased
+- fix(api-server): Preserve original host header for proxied API requests
+Some apps rely on reading the host header(eg multi-tenant apps served over multiple subdomains).  This change forwards on the original host header on proxied Fastify requests, and the experimental SSR/RSC server
 
 - fix(deps): update prisma monorepo to v5.10.2 (#10088)
 

diff --git a/packages/adapters/fastify/web/src/web.ts b/packages/adapters/fastify/web/src/web.ts
@@ -31,6 +31,13 @@ export async function redwoodFastifyWeb(
       prefix: redwoodOptions.apiUrl,
       upstream: redwoodOptions.apiProxyTarget,
       disableCache: true,
+      replyOptions: {
+        rewriteRequestHeaders: (req, headers) => ({
+          ...headers,
+          // preserve the original host header, instead of letting it be overwritten by the proxy
+          host: req.headers.host,
+        }),
+      },
     })
   }
 

diff --git a/packages/vite/src/runFeServer.ts b/packages/vite/src/runFeServer.ts
@@ -109,7 +109,7 @@ export async function runFeServer() {
     // @WARN! Be careful, between v2 and v3 of http-proxy-middleware
     // the syntax has changed https://github.com/chimurai/http-proxy-middleware
     createProxyMiddleware({
-      changeOrigin: true,
+      changeOrigin: false,
       pathRewrite: {
         [`^${rwConfig.web.apiUrl}`]: '', // remove base path
       },