diff options
author | Matthew Shao <me@matshao.com> | 2018-01-08 21:51:03 +0800 |
---|---|---|
committer | Matthew Shao <me@matshao.com> | 2018-01-08 21:51:03 +0800 |
commit | a6f6f8cd3226de3c89ec9d6cdeab972c88b94a18 (patch) | |
tree | 03cbac2b88606b348717a87a2cfc703672cac8c4 | |
parent | 633bc350d225eb02c7ab0aef6bae57c4def560e1 (diff) | |
download | mitmproxy-a6f6f8cd3226de3c89ec9d6cdeab972c88b94a18.tar.gz mitmproxy-a6f6f8cd3226de3c89ec9d6cdeab972c88b94a18.tar.bz2 mitmproxy-a6f6f8cd3226de3c89ec9d6cdeab972c88b94a18.zip |
Raise a CommandError for the exception
-rw-r--r-- | mitmproxy/addons/view.py | 3 | ||||
-rw-r--r-- | test/mitmproxy/addons/test_view.py | 7 |
2 files changed, 5 insertions, 5 deletions
diff --git a/mitmproxy/addons/view.py b/mitmproxy/addons/view.py index 68a3a72b..c3fb4b88 100644 --- a/mitmproxy/addons/view.py +++ b/mitmproxy/addons/view.py @@ -444,8 +444,7 @@ class View(collections.Sequence): try: req = http.HTTPRequest.make(method.upper(), url) except ValueError as e: - ctx.log.error(e) - return + raise exceptions.CommandError("Invalid URL: %s" % e) c = connections.ClientConnection.make_dummy(("", 0)) s = connections.ServerConnection.make_dummy((req.host, req.port)) f = http.HTTPFlow(c, s) diff --git a/test/mitmproxy/addons/test_view.py b/test/mitmproxy/addons/test_view.py index 72f1a0fb..a95d059d 100644 --- a/test/mitmproxy/addons/test_view.py +++ b/test/mitmproxy/addons/test_view.py @@ -147,9 +147,10 @@ def test_create(): assert v[0].request.url == "http://foo.com/" v.create("get", "http://foo.com") assert len(v) == 2 - v.create("get", "http://foo.com\\") - v.create("get", "http://") - assert len(v) == 2 + with pytest.raises(exceptions.CommandError, match="Invalid URL"): + v.create("get", "http://foo.com\\") + with pytest.raises(exceptions.CommandError, match="Invalid URL"): + v.create("get", "http://") def test_orders(): |