keep method when cloning Request, fixes #36522 (#36539)

keep method when cloning Request Co-authored-by: JJ Kasper <jj@jjsweb.site>
vercel · May 22, 2022 · 863db9b · 863db9b
1 parent 3bd42b4
commit 863db9b
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/packages/next/server/web/spec-compliant/request.ts b/packages/next/server/web/spec-compliant/request.ts
@@ -16,7 +16,9 @@ class BaseRequest extends Body implements Request {
   }
 
   constructor(input: BaseRequest | string, init: RequestInit = {}) {
-    const method = init.method?.toUpperCase() ?? 'GET'
+    const method =
+      init.method?.toUpperCase() ??
+      (input instanceof BaseRequest ? input.method?.toUpperCase() : 'GET')
 
     if (
       (method === 'GET' || method === 'HEAD') &&

diff --git a/test/unit/web-runtime/request.test.ts b/test/unit/web-runtime/request.test.ts
@@ -45,3 +45,14 @@ it('Request.referrer can be customized', async () => {
   const request = new Request('https://vercel.com', { referrer: 'client' })
   expect(request.referrer).toBe('client')
 })
+
+it('Request copies over method', async () => {
+  const request = new Request('https://vercel.com', {
+    method: 'POST',
+    body: 'hello',
+  })
+
+  const clonedRequest = new Request(request)
+
+  expect(clonedRequest.method).toBe(request.method)
+})