summaryrefslogtreecommitdiffstats
path: root/Lib
diff options
context:
space:
mode:
authorMartin Panter <vadmium+py@gmail.com>2016-04-30 01:30:57 (GMT)
committerMartin Panter <vadmium+py@gmail.com>2016-04-30 01:30:57 (GMT)
commit51b697b7f36e26c73136684f95e5d407fc2ed474 (patch)
tree66264b55b1807c6710187934d7e6bba119510c1e /Lib
parent0a5bd51dd30762e3e3b0f3b00ffe7899b9c58317 (diff)
parentaa27982ffca6ebfaad0a776e93e98a30e95b8b88 (diff)
downloadcpython-51b697b7f36e26c73136684f95e5d407fc2ed474.zip
cpython-51b697b7f36e26c73136684f95e5d407fc2ed474.tar.gz
cpython-51b697b7f36e26c73136684f95e5d407fc2ed474.tar.bz2
Issue #26864: Merge no_proxy fixes from 3.5
Diffstat (limited to 'Lib')
-rw-r--r--Lib/test/test_urllib.py13
-rw-r--r--Lib/urllib/request.py8
2 files changed, 19 insertions, 2 deletions
diff --git a/Lib/test/test_urllib.py b/Lib/test/test_urllib.py
index 3bf9ea7..49e2a2c 100644
--- a/Lib/test/test_urllib.py
+++ b/Lib/test/test_urllib.py
@@ -231,6 +231,19 @@ class ProxyTests(unittest.TestCase):
self.assertTrue(urllib.request.proxy_bypass_environment('anotherdomain.com:8888'))
self.assertTrue(urllib.request.proxy_bypass_environment('newdomain.com:1234'))
+ 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')
+ self.assertTrue(bypass('localhost'))
+ self.assertTrue(bypass('LocalHost')) # MixedCase
+ self.assertTrue(bypass('LOCALHOST')) # UPPERCASE
+ self.assertTrue(bypass('newdomain.com:1234'))
+ self.assertTrue(bypass('anotherdomain.com:8888'))
+ self.assertTrue(bypass('www.newdomain.com:1234'))
+ self.assertFalse(bypass('prelocalhost'))
+ self.assertFalse(bypass('newdomain.com')) # no port
+ self.assertFalse(bypass('newdomain.com:1235')) # wrong port
class ProxyTests_withOrderedEnv(unittest.TestCase):
diff --git a/Lib/urllib/request.py b/Lib/urllib/request.py
index e0a96e0..c519c2d 100644
--- a/Lib/urllib/request.py
+++ b/Lib/urllib/request.py
@@ -2492,8 +2492,12 @@ def proxy_bypass_environment(host, proxies=None):
# check if the host ends with any of the DNS suffixes
no_proxy_list = [proxy.strip() for proxy in no_proxy.split(',')]
for name in no_proxy_list:
- if name and (hostonly.endswith(name) or host.endswith(name)):
- return 1
+ if name:
+ name = re.escape(name)
+ pattern = r'(.+\.)?%s$' % name
+ if (re.match(pattern, hostonly, re.I)
+ or re.match(pattern, host, re.I)):
+ return 1
# otherwise, don't bypass
return 0