[PR #10564/a59e74b7 backport][3.11] Log offending websocket client address when no protocols overlap (#10575)

**This is a backport of PR #10564 as merged into master
(a59e74b).**



<!-- Thank you for your contribution! -->

## What do these changes do?

Logs the remote address of a WebSocket client that has no overlapping
protocols

## Are there changes in behavior for the user?

Which client has the problem should be a bit more discoverable

## Is it a substantial burden for the maintainers to support this?

no
## Related issue number
closes #10563

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

Author: patchback[bot]

CHANGES/10564.feature.rst

@@ -0,0 +1 @@
+Improved logging on non-overlapping WebSocket client protocols to include the remote address -- by :user:`bdraco`.

aiohttp/web_ws.py

@@ -252,7 +252,8 @@ def _handshake(
             else:
                 # No overlap found: Return no protocol as per spec
                 ws_logger.warning(
-                    "Client protocols %r don’t overlap server-known ones %r",
+                    "%s: Client protocols %r don’t overlap server-known ones %r",
+                    request.remote,
                     req_protocols,
                     self._protocols,
                 )

tests/test_websocket_handshake.py

@@ -174,7 +174,7 @@ async def test_handshake_protocol_unsupported(caplog) -> None:
 
     assert (
         caplog.records[-1].msg
-        == "Client protocols %r don’t overlap server-known ones %r"
+        == "%s: Client protocols %r don’t overlap server-known ones %r"
     )
     assert ws.ws_protocol is None