Add support for max_redirects in HTTPClient and corresponding unit tests

Author: HC-ONLINE

src/ciberwebscan/core/client/http_client.py

@@ -120,6 +120,7 @@ def __init__(
         metrics_callback: MetricsCallback | None = None,
         proxy: str | None = None,
         retryable_status_codes: set[int] | None = None,
+        max_redirects: int | None = None,
     ):
         """
         Initialize the HTTP client.
@@ -141,6 +142,8 @@ def __init__(
                 Supports http, https, and socks5 protocols.
             retryable_status_codes: HTTP status codes that trigger retry.
                 If None, uses config value or DEFAULT_RETRYABLE_STATUS_CODES.
+            max_redirects: Maximum number of redirects to follow.
+                If None, uses config value (default 10).
         """
         config = get_config().http
 
@@ -171,6 +174,9 @@ def __init__(
         resolved_follow_redirects = (
             config.follow_redirects if follow_redirects is None else follow_redirects
         )
+        resolved_max_redirects = (
+            config.max_redirects if max_redirects is None else max_redirects
+        )
 
         resolved_proxy = proxy
         if resolved_proxy is None and config.proxy is not None:
@@ -206,6 +212,7 @@ def __init__(
             http2=resolved_http2,
             verify=resolved_verify,
             follow_redirects=resolved_follow_redirects,
+            max_redirects=resolved_max_redirects,
             headers=default_headers,
             proxy=resolved_proxy,
         )

tests/unit/core/client/test_http_client.py

@@ -271,6 +271,18 @@ def test_retry_on_retryable_status(self, mock_request):
         assert response.status_code == 200
         assert mock_request.call_count == 2
 
+    def test_max_redirects_from_config(self):
+        """Test that max_redirects is loaded from config (default 10)."""
+        client = HTTPClient()
+        assert client._client.max_redirects == 10
+        client.close()
+
+    def test_max_redirects_override(self):
+        """Test that max_redirects can be overridden per instance."""
+        client = HTTPClient(max_redirects=5)
+        assert client._client.max_redirects == 5
+        client.close()
+
     def test_convenience_methods_exist(self):
         """Test that all convenience methods exist."""
         client = HTTPClient()