diff options
| author | Xiang Zhang <angwerzx@126.com> | 2017-01-09 03:47:55 (GMT) |
|---|---|---|
| committer | Xiang Zhang <angwerzx@126.com> | 2017-01-09 03:47:55 (GMT) |
| commit | 959ff7f1c6412fe768a29ffe1d355f126dc9e80c (patch) | |
| tree | fcdf0ada43bebc1074162f12683ffed169a701f4 /Lib/test/test_urllib.py | |
| parent | 8cbd46f19ffa8b1ac7220fcd1c791ab4cade8e10 (diff) | |
| download | cpython-959ff7f1c6412fe768a29ffe1d355f126dc9e80c.zip cpython-959ff7f1c6412fe768a29ffe1d355f126dc9e80c.tar.gz cpython-959ff7f1c6412fe768a29ffe1d355f126dc9e80c.tar.bz2 | |
Issue #29142: Fix suffixes in no_proxy handling in urllib.
In urllib.request, suffixes in no_proxy environment variable with
leading dots could match related hostnames again (e.g. .b.c matches a.b.c).
Patch by Milan Oberkirch.
Diffstat (limited to 'Lib/test/test_urllib.py')
| -rw-r--r-- | Lib/test/test_urllib.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Lib/test/test_urllib.py b/Lib/test/test_urllib.py index 247598a..1772399 100644 --- a/Lib/test/test_urllib.py +++ b/Lib/test/test_urllib.py @@ -247,11 +247,12 @@ class ProxyTests(unittest.TestCase): def test_proxy_bypass_environment_host_match(self): bypass = urllib.request.proxy_bypass_environment self.env.set('NO_PROXY', - 'localhost, anotherdomain.com, newdomain.com:1234') + 'localhost, anotherdomain.com, newdomain.com:1234, .d.o.t') self.assertTrue(bypass('localhost')) self.assertTrue(bypass('LocalHost')) # MixedCase self.assertTrue(bypass('LOCALHOST')) # UPPERCASE self.assertTrue(bypass('newdomain.com:1234')) + self.assertTrue(bypass('foo.d.o.t')) # issue 29142 self.assertTrue(bypass('anotherdomain.com:8888')) self.assertTrue(bypass('www.newdomain.com:1234')) self.assertFalse(bypass('prelocalhost')) |