Deprecate t.w.h.HTTPClient

src/twisted/web/http.py

@@ -31,6 +31,7 @@
     it, as in the HTTP 1.1 chunked I{Transfer-Encoding} (RFC 7230 section 4.1).
     This limits how much data may be buffered when decoding the line.
 """
+
 from __future__ import annotations
 
 __all__ = [
@@ -131,7 +132,7 @@
 from twisted.python import log
 from twisted.python.compat import nativeString, networkString
 from twisted.python.components import proxyForInterface
-from twisted.python.deprecate import deprecated
+from twisted.python.deprecate import deprecated, deprecatedModuleAttribute
 from twisted.python.failure import Failure
 from twisted.web._responses import (
     ACCEPTED,
@@ -771,6 +772,14 @@ def rawDataReceived(self, data):
             self.setLineMode(rest)
 
 
+deprecatedModuleAttribute(
+    Version("Twisted", "NEXT", 0, 0),
+    "Use twisted.web.client.Agent instead.",
+    __name__,
+    HTTPClient.__name__,
+)
+
+
 # response codes that must have empty bodies
 NO_BODY_CODES = (204, 304)
 

src/twisted/web/iweb.py

@@ -9,6 +9,7 @@
     L{IBodyProducer.length} to indicate that the length of the entity
     body is not known in advance.
 """
+
 from typing import TYPE_CHECKING, Callable, List, Optional
 
 from zope.interface import Attribute, Interface
@@ -595,15 +596,15 @@ def deliverBody(protocol):
         L{IPushProducer}.  The protocol's C{connectionLost} method will be
         called with:
 
-            - ResponseDone, which indicates that all bytes from the response
+            - L{ResponseDone}, which indicates that all bytes from the response
               have been successfully delivered.
 
-            - PotentialDataLoss, which indicates that it cannot be determined
+            - L{PotentialDataLoss}, which indicates that it cannot be determined
               if the entire response body has been delivered.  This only occurs
               when making requests to HTTP servers which do not set
               I{Content-Length} or a I{Transfer-Encoding} in the response.
 
-            - ResponseFailed, which indicates that some bytes from the response
+            - L{ResponseFailed}, which indicates that some bytes from the response
               were lost.  The C{reasons} attribute of the exception may provide
               more specific indications as to why.
         """

src/twisted/web/newsfragments/12158.removal

@@ -0,0 +1 @@
+twisted.web.http.HTTPClient is now deprecated in favor of twisted.web.client.Agent

src/twisted/web/test/test_http.py

@@ -5,7 +5,6 @@
 Test HTTP support.
 """
 
-
 import base64
 import calendar
 import random
@@ -20,6 +19,7 @@
 from zope.interface.verify import verifyObject
 
 import hamcrest
+from incremental import Version
 
 from twisted.internet import address
 from twisted.internet.error import ConnectionDone, ConnectionLost
@@ -4407,6 +4407,22 @@ def test_writeHeadersSanitizesLinearWhitespace(self):
             )
 
 
+class HTTPClientDeprecationTests(unittest.SynchronousTestCase):
+    """
+    Test that L{http.HTTPClient} is deprecated.
+    """
+
+    def test_deprecated(self):
+        """
+        Accessing L{http.HTTPClient} produces a deprecation warning.
+        """
+        self.getDeprecatedModuleAttribute(
+            "twisted.web.http",
+            "HTTPClient",
+            Version("Twisted", "NEXT", 0, 0),
+        )
+
+
 class HTTPClientSanitizationTests(unittest.SynchronousTestCase):
     """
     Test that L{http.HTTPClient} sanitizes its output.