[PR #11114/758738e backport][3.12] Downgrade connector close error to debug (#11115)

Author: bdraco

CHANGES/11114.misc.rst

@@ -0,0 +1 @@
+Downgraded the logging level for connector close errors from ERROR to DEBUG, as these are expected behavior with TLS 1.3 connections -- by :user:`bdraco`.

aiohttp/connector.py

@@ -1,6 +1,5 @@
 import asyncio
 import functools
-import logging
 import random
 import socket
 import sys
@@ -60,6 +59,7 @@
     set_exception,
     set_result,
 )
+from .log import client_logger
 from .resolver import DefaultResolver
 
 if sys.version_info >= (3, 12):
@@ -137,7 +137,7 @@ async def _wait_for_close(waiters: List[Awaitable[object]]) -> None:
     results = await asyncio.gather(*waiters, return_exceptions=True)
     for res in results:
         if isinstance(res, Exception):
-            logging.error("Error while closing connector: %r", res)
+            client_logger.debug("Error while closing connector: %r", res)
 
 
 class Connection:

tests/test_connector.py

@@ -2,6 +2,7 @@
 import asyncio
 import gc
 import hashlib
+import logging
 import platform
 import socket
 import ssl
@@ -309,6 +310,7 @@ async def test_close_with_exception_during_closing(
     loop: asyncio.AbstractEventLoop, caplog: pytest.LogCaptureFixture
 ) -> None:
     """Test that exceptions during connection closing are logged."""
+    caplog.set_level(logging.DEBUG)
     proto = create_mocked_conn()
 
     # Make the closed future raise an exception when awaited
@@ -327,7 +329,7 @@ async def test_close_with_exception_during_closing(
 
     # Check that the error was logged
     assert len(caplog.records) == 1
-    assert caplog.records[0].levelname == "ERROR"
+    assert caplog.records[0].levelname == "DEBUG"
     assert "Error while closing connector" in caplog.records[0].message
     assert "RuntimeError('Connection close failed')" in caplog.records[0].message