Fix IPv6 Zone ID decoding to correctly handle RFC 6874 %25 separator

CHANGES/1653.bugfix.rst

@@ -0,0 +1,5 @@
+Fixed the :attr:`~yarl.URL.host` property incorrectly returning the
+percent-encoded zone ID separator ``%25`` instead of decoding it to ``%``
+for IPv6 Zone ID URLs (e.g. ``http://[fe80::1%251]/`` now correctly exposes
+``.host`` as ``fe80::1%1`` per :rfc:`6874`)
+-- by :user:`rodrigobnogueira`.

tests/test_url.py

@@ -398,6 +398,14 @@ def test_ipv4_zone() -> None:
     assert url.raw_host == SplitResult(*url._val).hostname
 
 
+def test_ipv6_zone_rfc6874() -> None:
+    url = URL("http://[fe80::1%251]/")
+    assert url.raw_host == "fe80::1%251"
+    assert url.host == "fe80::1%1"
+    assert url.host_subcomponent == "[fe80::1%251]"
+    assert str(url) == "http://[fe80::1%251]/"
+
+
 def test_port_for_explicit_port() -> None:
     url = URL("http://example.com:8888")
     assert 8888 == url.port

yarl/_url.py

@@ -771,6 +771,9 @@ def host(self) -> str | None:
             return None
         if raw and raw[-1].isdigit() or ":" in raw:
             # IP addresses are never IDNA encoded
+            if "%25" in raw:
+                # Decode RFC 6874 zone ID separator
+                return raw.replace("%25", "%")
             return raw
         return _idna_decode(raw)
 
@@ -1557,7 +1560,10 @@ def _encode_host(host: str, validate_host: bool) -> str:
     # If the host ends with a digit or contains a colon, its likely
     # an IP address.
     if host and (host[-1].isdigit() or ":" in host):
-        raw_ip, sep, zone = host.partition("%")
+        if "%25" in host:
+            raw_ip, sep, zone = host.partition("%25")
+        else:
+            raw_ip, sep, zone = host.partition("%")
         # If it looks like an IP, we check with _ip_compressed_version
         # and fall-through if its not an IP address. This is a performance
         # optimization to avoid parsing IP addresses as much as possible
@@ -1585,8 +1591,8 @@ def _encode_host(host: str, validate_host: bool) -> str:
             # LRU to keep the cache size small
             host = ip.compressed
             if ip.version == 6:
-                return f"[{host}%{zone}]" if sep else f"[{host}]"
-            return f"{host}%{zone}" if sep else host
+                return f"[{host}{sep}{zone}]" if sep else f"[{host}]"
+            return f"{host}{sep}{zone}" if sep else host
 
     # IDNA encoding is slow, skip it for ASCII-only strings
     if host.isascii():