[Python-checkins] cpython (merge 3.6 -> default): Issue #29142: Merge 3.6.
xiang.zhang
python-checkins at python.org
Sun Jan 8 22:57:40 EST 2017
https://hg.python.org/cpython/rev/3954f8f6be3d
changeset: 106056:3954f8f6be3d
parent: 106052:ea064ff3c10f
parent: 106055:e90efd9f203e
user: Xiang Zhang <angwerzx at 126.com>
date: Mon Jan 09 11:52:10 2017 +0800
summary:
Issue #29142: Merge 3.6.
files:
Lib/test/test_urllib.py | 3 ++-
Lib/urllib/request.py | 1 +
Misc/NEWS | 4 ++++
3 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/Lib/test/test_urllib.py b/Lib/test/test_urllib.py
--- a/Lib/test/test_urllib.py
+++ b/Lib/test/test_urllib.py
@@ -247,11 +247,12 @@
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'))
diff --git a/Lib/urllib/request.py b/Lib/urllib/request.py
--- a/Lib/urllib/request.py
+++ b/Lib/urllib/request.py
@@ -2513,6 +2513,7 @@
no_proxy_list = [proxy.strip() for proxy in no_proxy.split(',')]
for name in no_proxy_list:
if name:
+ name = name.lstrip('.') # ignore leading dots
name = re.escape(name)
pattern = r'(.+\.)?%s$' % name
if (re.match(pattern, hostonly, re.I)
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -212,6 +212,10 @@
Library
-------
+- Issue #29142: 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.
+
- Issue #28961: Fix unittest.mock._Call helper: don't ignore the name parameter
anymore. Patch written by Jiajun Huang.
--
Repository URL: https://hg.python.org/cpython
More information about the Python-checkins
mailing list