Ticket #650: putnobody.patch

_cprequest.py

@@ -1 @@
-"""CherryPy core request/response handling."""
-
-import Cookie
-import os
@@ -643 +642 @@
-    
-    def process_body(self):
-        """Convert request.rfile into request.params (or request.body)."""
+        if not self.headers.get("Content-Length", ""):
+            # No Content-Length header supplied (or it's 0).
+            # If we went ahead and called cgi.FieldStorage, it would hang,
+            # since it cannot determine when to stop reading from the socket.
+            # See http://www.cherrypy.org/ticket/493.
+            # See also http://www.cherrypy.org/ticket/650.
+            # Note also that we expect any HTTP server to have decoded
+            # any message-body that had a transfer-coding, and we expect
+            # the HTTP server to have supplied a Content-Length header
+            # which is valid for the decoded entity-body.
+            return
+        
-        # FieldStorage only recognizes POST, so fake it.
-        methenv = {'REQUEST_METHOD': "POST"}
-        try:

wsgiserver/__init__.py

@@ -267 +267 @@
-        if read_chunked:
-            if not self.decode_chunked():
-                return
-        else:
-            cl = environ.get("CONTENT_LENGTH")
-            if method in ("POST", "PUT") and cl is None:
-                # No Content-Length header supplied. This will hang
-                # cgi.FieldStorage, since it cannot determine when to
-                # stop reading from the socket.
-                # See http://www.cherrypy.org/ticket/493.
-                self.simple_response("411 Length Required")
-                return
-        
-        # From PEP 333:
-        # "Servers and gateways that implement HTTP 1.1 must provide