Merge pull request #105 from bltb/master

Allow empty HTTP header value.
This commit is contained in:
Maximilian Hils 2015-11-17 11:19:33 +01:00
commit 4d17ddba38
2 changed files with 5 additions and 1 deletions

View file

@ -321,7 +321,7 @@ def _read_headers(rfile):
try:
name, value = line.split(b":", 1)
value = value.strip()
if not name or not value:
if not name:
raise ValueError()
ret.append([name, value])
except ValueError:

View file

@ -297,6 +297,10 @@ class TestReadHeaders(object):
with raises(HttpSyntaxException):
self._read(data)
def test_read_empty_value(self):
data = b"bar:"
headers = self._read(data)
assert headers.fields == [[b"bar", b""]]
def test_read_chunked():
req = treq(content=None)