[PR #8522/5a9e5006 backport][3.10] Restore AsyncResolver to be the de…

…fault resolver (#8523)

Co-authored-by: J. Nick Koston <[email protected]>

CHANGES/8522.misc.rst

@@ -0,0 +1,5 @@
+Restore :py:class:`~aiohttp.resolver.AsyncResolver` to be the default resolver. -- by :user:`bdraco`.
+
+:py:class:`~aiohttp.resolver.AsyncResolver` was disabled by default because
+of IPv6 compatibility issues. These issues have been resolved and
+:py:class:`~aiohttp.resolver.AsyncResolver` is again now the default resolver.

aiohttp/resolver.py

@@ -7,16 +7,16 @@
 
 __all__ = ("ThreadedResolver", "AsyncResolver", "DefaultResolver")
 
+
 try:
     import aiodns
 
-    # aiodns_default = hasattr(aiodns.DNSResolver, 'getaddrinfo')
+    aiodns_default = hasattr(aiodns.DNSResolver, "getaddrinfo")
 except ImportError:  # pragma: no cover
     aiodns = None  # type: ignore[assignment]
+    aiodns_default = False
 
 
-aiodns_default = False
-
 _NUMERIC_SOCKET_FLAGS = socket.AI_NUMERICHOST | socket.AI_NUMERICSERV
 _SUPPORTS_SCOPE_ID = sys.version_info >= (3, 9, 0)
 

tests/test_resolver.py

@@ -328,9 +328,11 @@ async def test_async_resolver_aiodns_not_present(loop, monkeypatch) -> None:
         AsyncResolver(loop=loop)
 
 
-def test_default_resolver() -> None:
-    # if getaddrinfo:
-    #     assert DefaultResolver is AsyncResolver
-    # else:
-    #     assert DefaultResolver is ThreadedResolver
+@pytest.mark.skipif(not getaddrinfo, reason="aiodns >=3.2.0 required")
+def test_aio_dns_is_default() -> None:
+    assert DefaultResolver is AsyncResolver
+
+
+@pytest.mark.skipif(getaddrinfo, reason="aiodns <3.2.0 required")
+def test_threaded_resolver_is_default() -> None:
     assert DefaultResolver is ThreadedResolver