check for body existance

unjs · Jul 24, 2023 · 97ab38e · 97ab38e
1 parent 28aff66
commit 97ab38e
Showing 1 changed file with 18 additions and 3 deletions.
diff --git a/src/event/event.ts b/src/event/event.ts
@@ -9,8 +9,16 @@ import {
 } from "../utils";
 import { H3Response } from "./response";
 
-const DOUBLE_SLASH_RE = /[/\\]{2,}/g; // TODO: Dedup from request.ts
-const RawBodySymbol = Symbol.for("h3RawBody"); // TODO: Dedup from body.ts
+// TODO: Dedup from request.ts
+const DOUBLE_SLASH_RE = /[/\\]{2,}/g;
+
+// TODO: Dedup from body.ts
+const PayloadMethods: Set<HTTPMethod> = new Set([
+  "PATCH",
+  "POST",
+  "PUT",
+  "DELETE",
+]);
 
 export interface NodeEventContext {
   req: NodeIncomingMessage;
@@ -47,6 +55,10 @@ export class H3Event implements Pick<FetchEvent, "respondWith"> {
     );
   }
 
+  get _hasBody() {
+    return PayloadMethods.has(this.method!);
+  }
+
   get path() {
     if (!this._path) {
       this._path = this._originalPath.replace(DOUBLE_SLASH_RE, "/");
@@ -89,7 +101,10 @@ export class H3Event implements Pick<FetchEvent, "respondWith"> {
   }
 
   get body() {
-    if (!this._body) {
+    if (!this._hasBody) {
+      return undefined;
+    }
+    if (this._body === undefined) {
       this._body = new ReadableStream({
         start: (controller) => {
           this.node.req.on("data", (chunk) => {