Reject whitespace inside normalized base URLs

Co-authored-by: Shri Sukhani <shrisukhani@users.noreply.github.com>

Author: cursoragent

hyperbrowser/config.py

@@ -36,6 +36,8 @@ def normalize_base_url(base_url: str) -> str:
             raise HyperbrowserError("base_url must not be empty")
         if "\n" in normalized_base_url or "\r" in normalized_base_url:
             raise HyperbrowserError("base_url must not contain newline characters")
+        if any(character.isspace() for character in normalized_base_url):
+            raise HyperbrowserError("base_url must not contain whitespace characters")
 
         parsed_base_url = urlparse(normalized_base_url)
         if (

tests/test_config.py

@@ -214,6 +214,11 @@ def test_client_config_rejects_empty_or_invalid_base_url():
     ):
         ClientConfig(api_key="test-key", base_url="https://example.local/\napi")
 
+    with pytest.raises(
+        HyperbrowserError, match="base_url must not contain whitespace characters"
+    ):
+        ClientConfig(api_key="test-key", base_url="https://example .local")
+
 
 def test_client_config_normalizes_headers_to_internal_copy():
     headers = {"X-Correlation-Id": "abc123"}
@@ -323,3 +328,8 @@ def test_client_config_normalize_base_url_validates_and_normalizes():
         HyperbrowserError, match="base_url must not contain newline characters"
     ):
         ClientConfig.normalize_base_url("https://example.local/\napi")
+
+    with pytest.raises(
+        HyperbrowserError, match="base_url must not contain whitespace characters"
+    ):
+        ClientConfig.normalize_base_url("https://example.local/\tapi")

tests/test_url_building.py

@@ -104,6 +104,12 @@ def test_client_build_url_rejects_runtime_invalid_base_url_changes():
         ):
             client._build_url("/session")
 
+        client.config.base_url = "https://example .local"
+        with pytest.raises(
+            HyperbrowserError, match="base_url must not contain whitespace characters"
+        ):
+            client._build_url("/session")
+
         client.config.base_url = "   "
         with pytest.raises(HyperbrowserError, match="base_url must not be empty"):
             client._build_url("/session")