diff options
author | Aldo Cortesi <aldo@nullcube.com> | 2013-03-03 22:13:23 +1300 |
---|---|---|
committer | Aldo Cortesi <aldo@nullcube.com> | 2013-03-03 22:13:23 +1300 |
commit | 5f0ad7b2a6b857419017e3e72062ab4e0e328238 (patch) | |
tree | a85eac79dac5efc4ceab339caa1a8b5499e15128 /test | |
parent | 5a050bb6b2b1a0bf05f4cd35d87e6f1d7a2608c0 (diff) | |
download | mitmproxy-5f0ad7b2a6b857419017e3e72062ab4e0e328238.tar.gz mitmproxy-5f0ad7b2a6b857419017e3e72062ab4e0e328238.tar.bz2 mitmproxy-5f0ad7b2a6b857419017e3e72062ab4e0e328238.zip |
Ensure that HTTP methods are ASCII.
Diffstat (limited to 'test')
-rw-r--r-- | test/test_http.py | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/test/test_http.py b/test/test_http.py index e98a891f..77cc2624 100644 --- a/test/test_http.py +++ b/test/test_http.py @@ -136,6 +136,7 @@ def test_parse_http_protocol(): def test_parse_init_connect(): assert http.parse_init_connect("CONNECT host.com:443 HTTP/1.0") + assert not http.parse_init_connect("C\xfeONNECT host.com:443 HTTP/1.0") assert not http.parse_init_connect("CONNECT \0host.com:443 HTTP/1.0") assert not http.parse_init_connect("CONNECT host.com:444444 HTTP/1.0") assert not http.parse_init_connect("bogus") @@ -155,6 +156,9 @@ def test_prase_init_proxy(): assert pa == "/test" assert httpversion == (1, 1) + u = "G\xfeET http://foo.com:8888/test HTTP/1.1" + assert not http.parse_init_proxy(u) + assert not http.parse_init_proxy("invalid") assert not http.parse_init_proxy("GET invalid HTTP/1.1") assert not http.parse_init_proxy("GET http://foo.com:8888/test foo/1.1") @@ -162,10 +166,14 @@ def test_prase_init_proxy(): def test_parse_init_http(): u = "GET /test HTTP/1.1" - m, u, httpversion= http.parse_init_http(u) + m, u, httpversion = http.parse_init_http(u) assert m == "GET" assert u == "/test" assert httpversion == (1, 1) + + u = "G\xfeET /test HTTP/1.1" + assert not http.parse_init_http(u) + assert not http.parse_init_http("invalid") assert not http.parse_init_http("GET invalid HTTP/1.1") assert not http.parse_init_http("GET /test foo/1.1") |