Support credentials in URL with empty user, e.g. http://:password@host (

aio-libs#6494)

CHANGES/6494.bugfix

@@ -0,0 +1 @@
+Support credentials in URL with empty user, e.g. http://:password@host

CONTRIBUTORS.txt

@@ -71,6 +71,7 @@ Chih-Yuan Chen
 Chris AtLee
 Chris Laws
 Chris Moore
+Chris Shucksmith
 Christopher Schmitt
 Claudiu Popa
 Colin Dunklau

aiohttp/helpers.py

@@ -178,9 +178,9 @@ def from_url(cls, url: URL, *, encoding: str = "latin1") -> Optional["BasicAuth"
         """Create BasicAuth from url."""
         if not isinstance(url, URL):
             raise TypeError("url should be yarl.URL instance")
-        if url.user is None:
+        if url.user is None and url.password is None:
             return None
-        return cls(url.user, url.password or "", encoding=encoding)
+        return cls(url.user or "", url.password or "", encoding=encoding)
 
     def encode(self) -> str:
         """Encode credentials."""

tests/test_client_request.py

@@ -389,6 +389,13 @@ def test_basic_auth_from_url(make_request: Any) -> None:
     assert "python.org" == req.host
 
 
+def test_basic_auth_no_user_from_url(make_request: Any) -> None:
+    req = make_request("get", "http://:1234@python.org")
+    assert "AUTHORIZATION" in req.headers
+    assert "Basic bmtpbToxMjM0" == req.headers["AUTHORIZATION"]
+    assert "python.org" == req.host
+
+
 def test_basic_auth_from_url_overridden(make_request: Any) -> None:
     req = make_request(
         "get", "http://garbage@python.org", auth=aiohttp.BasicAuth("nkim", "1234")