review

Author: sommeeeer

packages/open-next/src/overrides/converters/node.ts

@@ -31,13 +31,6 @@ const converter: Converter = {
     );
     const query = getQueryFromSearchParams(url.searchParams);
 
-    if (req.protocol === "http" && url.hostname === "localhost") {
-      // This is used internally by Next.js during redirects in server actions. We need to set it to the origin of the request.
-      process.env.__NEXT_PRIVATE_ORIGIN = url.origin;
-      // This is to make `next-auth` and other libraries that rely on this header to work locally out of the box.
-      headers["x-forwarded-proto"] = req.protocol;
-    }
-
     return {
       type: "core",
       method: req.method ?? "GET",

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

@@ -34,6 +34,12 @@ const wrapper: WrapperHandler = async (handler, converter) => {
   });
 
   app.all("*paths", async (req, res) => {
+    if (req.protocol === "http" && req.hostname === "localhost") {
+      // This is used internally by Next.js during redirects in server actions. We need to set it to the origin of the request.
+      process.env.__NEXT_PRIVATE_ORIGIN = `${req.protocol}://${req.hostname}`;
+      // This is to make `next-auth` and other libraries that rely on this header to work locally out of the box.
+      req.headers["x-forwarded-proto"] = req.protocol;
+    }
     const internalEvent = await converter.convertFrom(req);
     const streamCreator: StreamCreator = {
       writeHeaders: (prelude) => {