Fix inconsistent invalid URL handling (#1832)

Author: rodrigo.nogueira

httpx/_client.py

@@ -394,6 +394,8 @@ def _merge_url(self, url: URL | str) -> URL:
         to create the URL used for the outgoing request.
         """
         merge_url = URL(url)
+        if merge_url.scheme and not merge_url.host:
+            raise InvalidURL(f"Invalid URL '{url}': has scheme but missing host")
         if merge_url.is_relative_url:
             # To merge URLs we always append to the base URL. To get this
             # behaviour correct we always ensure the base URL ends in a '/'

tests/client/test_client.py

@@ -44,6 +44,21 @@ def test_get_invalid_url(server, url):
             client.get(url)
 
 
+def test_get_invalid_url_with_scheme_no_host():
+    """
+    Regression test for: https://github.com/encode/httpx/issues/1832
+    URLs with scheme but no host should raise InvalidURL.
+    """
+    with httpx.Client() as client:
+        with pytest.raises(httpx.InvalidURL) as exc:
+            client.get("https:/google.com")
+        assert "has scheme but missing host" in str(exc.value)
+
+        with pytest.raises(httpx.InvalidURL) as exc:
+            client.get("https:///google.com")
+        assert "has scheme but missing host" in str(exc.value)
+
+
 def test_build_request(server):
     url = server.url.copy_with(path="/echo_headers")
     headers = {"Custom-header": "value"}