changeset:   106055:e90efd9f203e
branch:      3.6
parent:      106049:35c1ca2aa994
parent:      106054:350c5a1f9cfd
user:        Xiang Zhang <angwerzx@126.com>
date:        Mon Jan 09 11:50:02 2017 +0800
files:       Lib/test/test_urllib.py Lib/urllib/request.py Misc/NEWS
description:
Issue #29142: Merge 3.5.


diff -r 35c1ca2aa994 -r e90efd9f203e Lib/test/test_urllib.py
--- a/Lib/test/test_urllib.py	Sun Jan 08 18:22:24 2017 -0800
+++ b/Lib/test/test_urllib.py	Mon Jan 09 11:50:02 2017 +0800
@@ -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 -r 35c1ca2aa994 -r e90efd9f203e Lib/urllib/request.py
--- a/Lib/urllib/request.py	Sun Jan 08 18:22:24 2017 -0800
+++ b/Lib/urllib/request.py	Mon Jan 09 11:50:02 2017 +0800
@@ -2514,6 +2514,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 -r 35c1ca2aa994 -r e90efd9f203e Misc/NEWS
--- a/Misc/NEWS	Sun Jan 08 18:22:24 2017 -0800
+++ b/Misc/NEWS	Mon Jan 09 11:50:02 2017 +0800
@@ -44,6 +44,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.