aboutsummaryrefslogtreecommitdiffstats
path: root/test
diff options
context:
space:
mode:
authorMaximilian Hils <git@maximilianhils.com>2016-11-20 17:21:21 +0100
committerThomas Kriechbaumer <Kriechi@users.noreply.github.com>2016-11-20 17:21:21 +0100
commitf74e561524d04c93cd7953f34e78ebe67eaa58a8 (patch)
treec1bf7ad5c3f583c4e25f11108826d514b1d36086 /test
parent79c753d8f8ea934c0e3d7973ba520c6e3d903d9f (diff)
downloadmitmproxy-f74e561524d04c93cd7953f34e78ebe67eaa58a8.tar.gz
mitmproxy-f74e561524d04c93cd7953f34e78ebe67eaa58a8.tar.bz2
mitmproxy-f74e561524d04c93cd7953f34e78ebe67eaa58a8.zip
rewrite absolute-form HTTP requests to relative form, fix #1759 (#1765)
Diffstat (limited to 'test')
-rw-r--r--test/mitmproxy/test_server.py15
1 files changed, 15 insertions, 0 deletions
diff --git a/test/mitmproxy/test_server.py b/test/mitmproxy/test_server.py
index 9429ab0f..a7d8cea5 100644
--- a/test/mitmproxy/test_server.py
+++ b/test/mitmproxy/test_server.py
@@ -282,6 +282,21 @@ class TestHTTP(tservers.HTTPProxyTest, CommonMixin):
assert d.content == b"bar"
self.master.addons.remove(s)
+ def test_first_line_rewrite(self):
+ """
+ If mitmproxy is a regular HTTP proxy, it must rewrite an absolute-form request like
+ GET http://example.com/foo HTTP/1.0
+ to
+ GET /foo HTTP/1.0
+ when sending the request upstream. While any server should technically accept
+ the absolute form, this is not the case in practice.
+ """
+ req = "get:'%s/p/200'" % self.server.urlbase
+ p = self.pathoc()
+ with p.connect():
+ assert p.request(req).status_code == 200
+ assert self.server.last_log()["request"]["first_line_format"] == "relative"
+
class TestHTTPAuth(tservers.HTTPProxyTest):
def test_auth(self):