fix: handle redirect response body (#142)

* fix: handle redirect response body

* Use js

* Update src/run/handlers/server.ts

Co-authored-by: Eduardo Bouças <[email protected]>

* Fix test

---------

Co-authored-by: Eduardo Bouças <[email protected]>

Author: ascorbic

src/run/handlers/server.ts

@@ -58,5 +58,12 @@ export default async (request: Request) => {
   setCacheTagsHeaders(response.headers, request, tagsManifest)
   setVaryHeaders(response.headers, request, nextConfig)
 
+  // Temporary workaround for an issue where sending a response with an empty
+  // body causes an unhandled error.
+  // TODO: Remove once a fix has been rolled out.
+  if (response.status > 300 && response.status < 400) {
+    return new Response(null, response)
+  }
+
   return response
 }

tests/e2e/simple-app.test.ts

@@ -32,3 +32,11 @@ test('Serves a static image correctly', async ({ page }) => {
   expect(response?.status()).toBe(200)
   expect(response?.headers()['content-type']).toBe('image/svg+xml')
 })
+
+test('Redirects correctly', async ({ page }) => {
+  await page.goto(`${ctx.url}/redirect/response`)
+  await expect(page).toHaveURL(`https://www.netlify.com/`)
+
+  await page.goto(`${ctx.url}/redirect`)
+  await expect(page).toHaveURL(`https://www.netlify.com/`)
+})

tests/fixtures/simple-next-app/app/redirect/response/route.js

@@ -0,0 +1,5 @@
+import { NextRequest, NextResponse } from 'next/server'
+
+export async function GET() {
+  return NextResponse.redirect('https://www.netlify.com/')
+}

tests/fixtures/simple-next-app/app/redirect/route.js

@@ -0,0 +1,5 @@
+import { redirect } from 'next/navigation'
+
+export async function GET() {
+  return redirect('https://www.netlify.com/')
+}

tests/integration/simple-app.test.ts

@@ -41,6 +41,8 @@ test<FixtureTestContext>('Test that the simple next app is working', async (ctx)
     '500.html',
     'index',
     'other',
+    'redirect',
+    'redirect/response',
   ])
 
   // test the function call