Handle :status header better.

This resolves #16.

Author: Lukasa

hyper/http20/response.py

@@ -19,6 +19,10 @@ def __init__(self, headers, stream):
         #: HTTP/2.0, and so is always the empty string.
         self.reason = ''
 
+        #: The status code returned by the server.
+        self.status = int(headers[':status'])
+        del headers[':status']
+
         # The response headers. These are determined upon creation, assigned
         # once, and never assigned again.
         # This conversion to dictionary is unwise, as there may be repeated
@@ -70,10 +74,3 @@ def fileno(self):
         Return the ``fileno`` of the underlying socket.
         """
         pass
-
-    @property
-    def status(self):
-        """
-        Status code returned by the server.
-        """
-        return int(self._headers[':status'])

test/test_hyper.py

@@ -11,6 +11,7 @@
 from hyper.http20.stream import (
     Stream, STATE_HALF_CLOSED_LOCAL, STATE_OPEN, MAX_CHUNK, STATE_CLOSED
 )
+from hyper.http20.response import HTTP20Response
 import pytest
 from io import BytesIO
 
@@ -1022,6 +1023,16 @@ def inner():
         assert len(out_frames) == 1
         assert s.state == STATE_CLOSED
 
+
+class TestResponse(object):
+    def test_status_is_stripped_from_headers(self):
+        headers = {':status': '200'}
+        resp = HTTP20Response(headers, None)
+
+        assert resp.status == 200
+        assert resp.getheaders() == []
+
+
 # Some utility classes for the tests.
 class NullEncoder(object):
     def encode(headers):