]> granicus.if.org Git - python/commitdiff
Issue #18792: Use "127.0.0.1" or "::1" instead of "localhost" as much as possible...
authorAntoine Pitrou <solipsis@pitrou.net>
Wed, 21 Aug 2013 22:39:46 +0000 (00:39 +0200)
committerAntoine Pitrou <solipsis@pitrou.net>
Wed, 21 Aug 2013 22:39:46 +0000 (00:39 +0200)
Lib/test/_test_multiprocessing.py
Lib/test/support/__init__.py
Lib/test/test_asyncore.py
Lib/test/test_ftplib.py
Lib/test/test_timeout.py
Misc/NEWS

index f9be81064931281daaca6a065eb20a07cbaab006..d1c7f0ba00e0a921f4f20e7f34d91ba5989a373a 100644 (file)
@@ -1999,7 +1999,7 @@ class _TestRemoteManager(BaseTestCase):
         authkey = os.urandom(32)
 
         manager = QueueManager(
-            address=('localhost', 0), authkey=authkey, serializer=SERIALIZER
+            address=(test.support.HOST, 0), authkey=authkey, serializer=SERIALIZER
             )
         manager.start()
 
@@ -2037,7 +2037,7 @@ class _TestManagerRestart(BaseTestCase):
     def test_rapid_restart(self):
         authkey = os.urandom(32)
         manager = QueueManager(
-            address=('localhost', 0), authkey=authkey, serializer=SERIALIZER)
+            address=(test.support.HOST, 0), authkey=authkey, serializer=SERIALIZER)
         srvr = manager.get_server()
         addr = srvr.address
         # Close the connection.Listener socket which gets opened as a part
@@ -2509,7 +2509,7 @@ class _TestPicklingConnections(BaseTestCase):
             l.close()
 
         l = socket.socket()
-        l.bind(('localhost', 0))
+        l.bind((test.support.HOST, 0))
         l.listen(1)
         conn.send(l.getsockname())
         new_conn, addr = l.accept()
@@ -3151,9 +3151,9 @@ class TestWait(unittest.TestCase):
     def test_wait_socket(self, slow=False):
         from multiprocessing.connection import wait
         l = socket.socket()
-        l.bind(('', 0))
+        l.bind((test.support.HOST, 0))
         l.listen(4)
-        addr = ('localhost', l.getsockname()[1])
+        addr = l.getsockname()
         readers = []
         procs = []
         dic = {}
index befbe8bee2fe162301e1b1d6f28adc0152c7b4e8..02ea2987a078e3d910d7f584e96267cd21b1d7eb 100644 (file)
@@ -480,7 +480,11 @@ def requires_mac_ver(*min_version):
     return decorator
 
 
-HOST = 'localhost'
+# Don't use "localhost", since resolving it uses the DNS under recent
+# Windows versions (see issue #18792).
+HOST = "127.0.0.1"
+HOSTv6 = "::1"
+
 
 def find_unused_port(family=socket.AF_INET, socktype=socket.SOCK_STREAM):
     """Returns an unused port that should be suitable for binding.  This is
index c02a97689ef6f1df40013be9d8feda226a3f2e1e..87655f4495fa8a498ef7a666a1f09c6230e0b372 100644 (file)
@@ -10,7 +10,7 @@ import errno
 import struct
 
 from test import support
-from test.support import TESTFN, run_unittest, unlink
+from test.support import TESTFN, run_unittest, unlink, HOST, HOSTv6
 from io import BytesIO
 from io import StringIO
 
@@ -19,7 +19,6 @@ try:
 except ImportError:
     threading = None
 
-HOST = support.HOST
 TIMEOUT = 3
 HAS_UNIX_SOCKETS = hasattr(socket, 'AF_UNIX')
 
@@ -816,7 +815,7 @@ class TestAPI_UseIPv4Sockets(BaseTestAPI):
 @unittest.skipUnless(support.IPV6_ENABLED, 'IPv6 support required')
 class TestAPI_UseIPv6Sockets(BaseTestAPI):
     family = socket.AF_INET6
-    addr = ('::1', 0)
+    addr = (HOSTv6, 0)
 
 @unittest.skipUnless(HAS_UNIX_SOCKETS, 'Unix sockets required')
 class TestAPI_UseUnixSockets(BaseTestAPI):
index 25e34be2d82efbd89755a4f2592af70caa85d20d..08d99d80dea2cee0314c4914c7d979901e84a7c5 100644 (file)
@@ -18,7 +18,7 @@ except ImportError:
 
 from unittest import TestCase
 from test import support
-from test.support import HOST
+from test.support import HOST, HOSTv6
 threading = support.import_module('threading')
 
 TIMEOUT = 3
@@ -767,7 +767,7 @@ class TestFTPClass(TestCase):
 class TestIPv6Environment(TestCase):
 
     def setUp(self):
-        self.server = DummyFTPServer(('::1', 0), af=socket.AF_INET6)
+        self.server = DummyFTPServer((HOSTv6, 0), af=socket.AF_INET6)
         self.server.start()
         self.client = ftplib.FTP(timeout=TIMEOUT)
         self.client.connect(self.server.host, self.server.port)
@@ -950,7 +950,7 @@ class TestTimeouts(TestCase):
         self.assertTrue(socket.getdefaulttimeout() is None)
         socket.setdefaulttimeout(30)
         try:
-            ftp = ftplib.FTP("localhost")
+            ftp = ftplib.FTP(HOST)
         finally:
             socket.setdefaulttimeout(None)
         self.assertEqual(ftp.sock.gettimeout(), 30)
@@ -962,7 +962,7 @@ class TestTimeouts(TestCase):
         self.assertTrue(socket.getdefaulttimeout() is None)
         socket.setdefaulttimeout(30)
         try:
-            ftp = ftplib.FTP("localhost", timeout=None)
+            ftp = ftplib.FTP(HOST, timeout=None)
         finally:
             socket.setdefaulttimeout(None)
         self.assertTrue(ftp.sock.gettimeout() is None)
index fe7b264c6cdee11766e359cc003f42fe158fb123..703c43ab2bc7b7c1741f6bb325397b6c5058e061 100644 (file)
@@ -110,7 +110,7 @@ class TimeoutTestCase(unittest.TestCase):
     # solution.
     fuzz = 2.0
 
-    localhost = '127.0.0.1'
+    localhost = support.HOST
 
     def setUp(self):
         raise NotImplementedError()
index dff9740b0ca02428b2308086e5d1316551aebef9..61f5369584220feccda4eda6f864e9a1c47d94be 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -110,6 +110,13 @@ Library
 
 - Issue #8860: Fixed rounding in timedelta constructor.
 
+Tests
+-----
+
+- Issue #18792: Use "127.0.0.1" or "::1" instead of "localhost" as much as
+  possible, since "localhost" goes through a DNS lookup under recent Windows
+  versions.
+
 
 What's New in Python 3.4.0 Alpha 1?
 ===================================