Allow undefined body sent to sendData() (#20981)

Co-authored-by: rokobekavac0 <roko.beakavac2003@gmail.com> Co-authored-by: Tim Neutkens <tim@timneutkens.nl> Co-authored-by: Joe Haddad <joe.haddad@zeit.co>
vercel · Jan 26, 2021 · 004ad62 · 004ad62
1 parent 07d4af9
commit 004ad62
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/packages/next/next-server/server/api-utils.ts b/packages/next/next-server/server/api-utils.ts
@@ -231,7 +231,7 @@ export function sendData(
   res: NextApiResponse,
   body: any
 ): void {
-  if (body === null) {
+  if (body === null || body === undefined) {
     res.end()
     return
   }

diff --git a/test/integration/api-support/pages/api/undefined.js b/test/integration/api-support/pages/api/undefined.js
@@ -0,0 +1,3 @@
+export default (req, res) => {
+  res.json(undefined)
+}
diff --git a/test/integration/api-support/test/index.test.js b/test/integration/api-support/test/index.test.js
@@ -122,6 +122,12 @@ function runTests(dev = false) {
     expect(body).toBe(true)
   })
 
+  it('should support undefined response body', async () => {
+    const res = await fetchViaHTTP(appPort, '/api/undefined', null, {})
+    const body = res.ok ? await res.text() : null
+    expect(body).toBe('')
+  })
+
   it('should return error with invalid JSON', async () => {
     const data = await fetchViaHTTP(appPort, '/api/parse', null, {
       method: 'POST',