just boolean

Author: ijjk

Diff for: packages/next/src/server/lib/router-server.ts

@@ -325,7 +325,7 @@ export async function initialize(opts: {
 
       // handle hot-reloader first
       if (developmentBundler) {
-        if (blockCrossSite(req, res, allowedOrigins, `${opts.port}`).finished) {
+        if (blockCrossSite(req, res, allowedOrigins, `${opts.port}`)) {
           return
         }
         const origUrl = req.url || '/'
@@ -691,9 +691,7 @@ export async function initialize(opts: {
       })
 
       if (opts.dev && developmentBundler && req.url) {
-        if (
-          blockCrossSite(req, socket, allowedOrigins, `${opts.port}`).finished
-        ) {
+        if (blockCrossSite(req, socket, allowedOrigins, `${opts.port}`)) {
           return
         }
         const { basePath, assetPrefix } = config

Diff for: packages/next/src/server/lib/router-utils/block-cross-site.ts

@@ -8,12 +8,10 @@ export const blockCrossSite = (
   res: ServerResponse | Duplex,
   allowedOrigins: string[],
   activePort: string
-): {
-  finished?: boolean
-} => {
+): boolean => {
   // only process _next URLs
   if (!req.url?.includes('/_next')) {
-    return {}
+    return false
   }
   // block non-cors request from cross-site e.g. script tag on
   // different host
@@ -25,9 +23,7 @@ export const blockCrossSite = (
       res.statusCode = 403
     }
     res.end('Unauthorized')
-    return {
-      finished: true,
-    }
+    return true
   }
 
   // ensure websocket requests from allowed origin
@@ -54,12 +50,10 @@ export const blockCrossSite = (
           res.statusCode = 403
         }
         res.end('Unauthorized')
-        return {
-          finished: true,
-        }
+        return true
       }
     }
   }
 
-  return {}
+  return false
 }