]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
gh-100374: Fixed a bug in socket.getfqdn() (gh-100375)
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Wed, 21 Dec 2022 14:06:10 +0000 (06:06 -0800)
committerGitHub <noreply@github.com>
Wed, 21 Dec 2022 14:06:10 +0000 (06:06 -0800)
(cherry picked from commit 12be23cf3c1301be2c6b8fd4cb2cd35a567d2ea2)

Co-authored-by: Dominic Socular <BBH@awsl.rip>
Lib/socket.py
Lib/test/test_socket.py
Misc/NEWS.d/next/Core and Builtins/2022-12-20-16-14-19.gh-issue-100374.YRrVHT.rst [new file with mode: 0644]

index 0717c696b12e036a2c6bcd3265cded966589a832..5a896bee7c49bb6ec4da3e0e3af2cf20ea9db06f 100644 (file)
@@ -784,11 +784,11 @@ def getfqdn(name=''):
 
     First the hostname returned by gethostbyaddr() is checked, then
     possibly existing aliases. In case no FQDN is available and `name`
-    was given, it is returned unchanged. If `name` was empty or '0.0.0.0',
+    was given, it is returned unchanged. If `name` was empty, '0.0.0.0' or '::',
     hostname from gethostname() is returned.
     """
     name = name.strip()
-    if not name or name == '0.0.0.0':
+    if not name or name in ('0.0.0.0', '::'):
         name = gethostname()
     try:
         hostname, aliases, ipaddrs = gethostbyaddr(name)
index b07954989fdd87e6d6a8cfa882bff936a4c3382e..13cb2a7945d1db32cb4805155fc9d16139ecc3ad 100644 (file)
@@ -1760,6 +1760,10 @@ class GeneralModuleTests(unittest.TestCase):
         )
         self.assertEqual(sockaddr, ('ff02::1de:c0:face:8d', 1234, 0, 0))
 
+    def test_getfqdn_filter_localhost(self):
+        self.assertEqual(socket.getfqdn(), socket.getfqdn("0.0.0.0"))
+        self.assertEqual(socket.getfqdn(), socket.getfqdn("::"))
+
     @unittest.skipUnless(socket_helper.IPV6_ENABLED, 'IPv6 required for this test.')
     @unittest.skipIf(sys.platform == 'win32', 'does not work on Windows')
     @unittest.skipIf(AIX, 'Symbolic scope id does not work')
diff --git a/Misc/NEWS.d/next/Core and Builtins/2022-12-20-16-14-19.gh-issue-100374.YRrVHT.rst b/Misc/NEWS.d/next/Core and Builtins/2022-12-20-16-14-19.gh-issue-100374.YRrVHT.rst
new file mode 100644 (file)
index 0000000..e78352f
--- /dev/null
@@ -0,0 +1 @@
+Fix incorrect result and delay in :func:`socket.getfqdn`. Patch by Dominic Socular.