Address review feedback from picnixz

Author: JackDanger

Lib/http/client.py

@@ -296,8 +296,8 @@ def __init__(self, sock, debuglevel=0, method=None, url=None):
 
     def __repr__(self):
         if self.status is _UNKNOWN:
-            return '<%s>' % (self.__class__.__name__,)
-        return '<%s [%s %s]>' % (self.__class__.__name__, self.status, self.reason)
+            return f'<{self.__class__.__name__}>'
+        return f'<{self.__class__.__name__} [{self.status} {self.reason}]>'
 
     def _read_status(self):
         line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")
@@ -917,7 +917,8 @@ def __init__(self, host, port=None, timeout=socket._GLOBAL_DEFAULT_TIMEOUT,
         self._create_connection = socket.create_connection
 
     def __repr__(self):
-        return '<%s %s:%s>' % (self.__class__.__name__, self.host, self.port if self.port is not None else self.default_port)
+        port = self.port if self.port is not None else self.default_port
+        return f'<{self.__class__.__name__} {self.host}:{port}>'
 
     def set_tunnel(self, host, port=None, headers=None):
         """Set up host and port for HTTP CONNECT tunnelling.

Lib/test/test_httplib.py

@@ -2373,15 +2373,15 @@ def test_http_connection_repr_explicit_port(self):
         conn = client.HTTPConnection('example.com', 8080)
         self.assertEqual(repr(conn), '<HTTPConnection example.com:8080>')
 
+    @unittest.skipUnless(hasattr(client, 'HTTPSConnection'),
+                         'ssl support required')
     def test_https_connection_repr(self):
-        if not hasattr(client, 'HTTPSConnection'):
-            self.skipTest('ssl support required')
         conn = client.HTTPSConnection('example.com')
         self.assertEqual(repr(conn), '<HTTPSConnection example.com:443>')
 
+    @unittest.skipUnless(hasattr(client, 'HTTPSConnection'),
+                         'ssl support required')
     def test_https_connection_repr_explicit_port(self):
-        if not hasattr(client, 'HTTPSConnection'):
-            self.skipTest('ssl support required')
         conn = client.HTTPSConnection('example.com', 8443)
         self.assertEqual(repr(conn), '<HTTPSConnection example.com:8443>')
 

Misc/NEWS.d/next/Library/2026-03-08-00-00-00.gh-issue-145651.HtRepr.rst

@@ -1,3 +1,3 @@
-Add :func:`repr` support to :class:`http.client.HTTPConnection` and
+Add :meth:`~object.__repr__` support to :class:`http.client.HTTPConnection` and
 :class:`http.client.HTTPResponse`, showing host/port and status/reason
 respectively.