Restore unquoting of %2F in paths (#1151)

aio-libs · Sep 23, 2024 · 497752c · 497752c
1 parent 6795a59
commit 497752c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/CHANGES/1151.breaking.rst b/CHANGES/1151.breaking.rst
@@ -0,0 +1,3 @@
+Restore decoding ``%2F`` (``/``) in ``URL.path`` -- by :user:`bdraco`.
+
+This change restored the behavior before :issue:`1136`.
diff --git a/tests/test_url.py b/tests/test_url.py
@@ -346,10 +346,10 @@ def test_path_with_spaces():
 
 
 def test_path_with_2F():
-    """Path should not decode %2F, otherwise it may look like a path separator."""
+    """Path should decode %2F."""
 
     url = URL("http://example.com/foo/bar%2fbaz")
-    assert url.path == "/foo/bar%2Fbaz"
+    assert url.path == "/foo/bar/baz"
 
 
 def test_raw_path_for_empty_url():

diff --git a/yarl/_url.py b/yarl/_url.py
@@ -225,7 +225,7 @@ class URL:
     _FRAGMENT_REQUOTER = _Quoter(safe="?/:@")
 
     _UNQUOTER = _Unquoter()
-    _PATH_UNQUOTER = _Unquoter(ignore="/", unsafe="+")
+    _PATH_UNQUOTER = _Unquoter(unsafe="+")
     _QS_UNQUOTER = _Unquoter(qs=True)
 
     _val: SplitResult