Only set up 'body' if there is one

promised-io · Mar 13, 2012 · 122cfb8 · 122cfb8
1 parent 0e41edc
commit 122cfb8
Showing 1 changed file with 12 additions and 9 deletions.
diff --git a/Request.js b/Request.js
@@ -34,18 +34,21 @@ define([
       headers: {
         enumerable: true,
         value: req.headers
-      },
+      }
+    });
 
-      /**
-      * node-http-server.Request#body -> node-stream.Stream
-      *
-      * A stream for the request body.
-      **/
-      body: {
+    /**
+    * node-http-server.Request#body -> node-stream.Stream
+    *
+    * A stream for the request body, unless the request method is GET, DELETE
+    * or HEAD.
+    **/
+    if(req.method !== "GET" && req.method !== "DELETE" && req.method !== "HEAD"){
+      Object.defineProperty(this, "body", {
         enumerable: true,
         value: new Stream(req)
-      }
-    });
+      });
+    }
 
     /**
     * node-http-server.Request#expectContinue -> promise.Deferred