refactor(event): always normalize event.method

unjs · Aug 1, 2023 · 7585861 · 7585861
1 parent a26579f
commit 7585861
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/src/event/event.ts b/src/event/event.ts
@@ -73,8 +73,13 @@ export class H3Event<
     );
   }
 
-  get method(): HTTPMethod | undefined {
-    return this._method || (this.node.req.method as HTTPMethod);
+  get method(): HTTPMethod {
+    if (!this._method) {
+      this._method = (
+        this.node.req.method || "GET"
+      ).toUpperCase() as HTTPMethod;
+    }
+    return this._method;
   }
 
   get headers(): Headers {

diff --git a/src/utils/proxy.ts b/src/utils/proxy.ts
@@ -38,7 +38,7 @@ export async function proxyRequest(
   // Request Body
   let body;
   let duplex: Duplex | undefined;
-  if (PayloadMethods.has(event.method!)) {
+  if (PayloadMethods.has(event.method)) {
     if (opts.streamRequest) {
       body = event.body;
       duplex = "half";
@@ -48,7 +48,7 @@ export async function proxyRequest(
   }
 
   // Method
-  const method = opts.fetchOptions?.method || event.method!;
+  const method = opts.fetchOptions?.method || event.method;
 
   // Headers
   const fetchHeaders = mergeHeaders(