Merge pull request #19 from w3c/jgraham/read_whole_request

Ensure that all of the request data is read from the socket when using a keepalive connection.

wptserve/server.py

@@ -216,6 +216,7 @@ def handle_one_request(self):
                     response.set_error(500, err)
             logger.info("%i %s %s (%s) %i" % (response.status[0], request.method,
                                          request.request_path, request.headers.get('Referer'), request.raw_input.length))
+
             if not response.writer.content_written:
                 response.write()
 
@@ -224,7 +225,11 @@ def handle_one_request(self):
             # can't set a Content-Length header. A notable example of this kind of
             # problem is with the trickle pipe i.e. foo.js?pipe=trickle(d1)
             if response.close_connection:
-                self.close_connection = 1
+                self.close_connection = True
+
+            if not self.close_connection:
+                # Ensure that the whole request has been read from the socket
+                request.raw_input.read()
 
         except socket.timeout, e:
             self.log_error("Request timed out: %r", e)