Merge pull request #325 from ex3me0/development

HTTP/1.1 parser fix, when response contains empty reason

Author: Lukasa

hyper/http11/parser.py

@@ -49,7 +49,8 @@ def parse_response(self, buffer):
         if index == -1:
             return None
 
-        version, status, reason = temp_buffer[0:index].split(None, 2)
+        version, status, reason = (
+                temp_buffer[0:index].split(None, 2) + [b''])[:3]
         if not version.startswith(b'HTTP/1.'):
             raise ParseError("Not HTTP/1.X!")
 

test/test_http11.py

@@ -329,6 +329,23 @@ def test_chunked_overrides_body(self):
 
         assert received == expected
 
+    def test_response_with_empty_reason(self):
+        c = HTTP11Connection('httpbin.org')
+        c._sock = sock = DummySocket()
+
+        sock._buffer = BytesIO(
+            b"HTTP/1.1 201 \r\n"
+            b"Connection: close\r\n"
+            b"Server: Socket\r\n"
+            b"Content-Length: 0\r\n"
+            b"\r\n"
+        )
+
+        r = c.get_response()
+
+        assert r.status == 201
+        assert r.reason == b''
+
     def test_get_response(self):
         c = HTTP11Connection('httpbin.org')
         c._sock = sock = DummySocket()