bpo-27929: resolve names only for AF_INET/AF_INET6 with asyncio (pythonGH-32131)

miss-islington · vincentbernat · web-flow · commit f84fb5565907 · 2022-03-28T15:16:30.000-07:00
Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com> (cherry picked from commit 5c30388) Co-authored-by: Vincent Bernat <vincent@bernat.ch>
diff --git a/Lib/asyncio/selector_events.py b/Lib/asyncio/selector_events.py
@@ -487,7 +487,8 @@ async def sock_connect(self, sock, address):
         if self._debug and sock.gettimeout() != 0:
             raise ValueError("the socket must be non-blocking")
 
-        if not hasattr(socket, 'AF_UNIX') or sock.family != socket.AF_UNIX:
+        if sock.family == socket.AF_INET or (
+                base_events._HAS_IPv6 and sock.family == socket.AF_INET6):
             resolved = await self._ensure_resolved(
                 address, family=sock.family, type=sock.type, proto=sock.proto,
                 loop=self,
diff --git a/Misc/NEWS.d/next/Library/2022-03-28-13-35-50.bpo-27929.j5mAmV.rst b/Misc/NEWS.d/next/Library/2022-03-28-13-35-50.bpo-27929.j5mAmV.rst
@@ -0,0 +1,3 @@
+Fix :meth:`asyncio.loop.sock_connect` to only resolve names for :const:`socket.AF_INET` or
+:const:`socket.AF_INET6` families. Resolution may not make sense for other families,
+like :const:`socket.AF_BLUETOOTH` and :const:`socket.AF_UNIX`.

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	+Fix :meth:`asyncio.loop.sock_connect` to only resolve names for :const:`socket.AF_INET` or
	`2`	+:const:`socket.AF_INET6` families. Resolution may not make sense for other families,
	`3`	+like :const:`socket.AF_BLUETOOTH` and :const:`socket.AF_UNIX`.