Skip to content

Cleanup URL percent-encoding behavior. #2990

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lovelydinosaur merged 13 commits into from
Dec 15, 2023
+113 −42
Merged

Cleanup URL percent-encoding behavior. #2990

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
9c669b7
Replace path_query_fragment encoding tests
lovelydinosaur Dec 7, 2023
6cc7555
Remove replaced test cases
lovelydinosaur Dec 7, 2023
b7d4258
Fix test case to use correct hex sequence for 'abc'
lovelydinosaur Dec 7, 2023
412a880
Fix 'quote' behaviour so we don't double-escape.
lovelydinosaur Dec 7, 2023
a1b4afc
Add '/' to safe chars in query strings
lovelydinosaur Dec 7, 2023
de9968a
Update docstring
lovelydinosaur Dec 11, 2023
7bdf227
Linting
lovelydinosaur Dec 11, 2023
76ec6eb
Merge branch 'master' into cleanup-url-encoding
lovelydinosaur Dec 11, 2023
dd7b657
Merge branch 'master' into cleanup-url-encoding
karpetrosyan Dec 13, 2023
0d8cc4b
Update outdated comment.
lovelydinosaur Dec 13, 2023
e3532aa
Revert unrelated change
lovelydinosaur Dec 13, 2023
84f6338
Merge branch 'master' into cleanup-url-encoding
lovelydinosaur Dec 13, 2023
53d15f1
Merge branch 'master' into cleanup-url-encoding
lovelydinosaur Dec 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update outdated comment.
  • Loading branch information
@lovelydinosaur
lovelydinosaur authored Dec 13, 2023
commit 0d8cc4b001e46e290550e02862a4d040bce0be43
2 changes: 0 additions & 2 deletions httpx/_urlparse.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -260,8 +260,6 @@ def urlparse(url: str = "", **kwargs: typing.Optional[str]) -> ParseResult:
# For 'path' we need to drop ? and # from the GEN_DELIMS set.
parsed_path: str = quote(path, safe=SUB_DELIMS + ":/[]@")
# For 'query' we need to drop '#' from the GEN_DELIMS set.
# We also exclude '/' because it is more robust to replace it with a percent
# encoding despite it not being a requirement of the spec.
parsed_query: typing.Optional[str] = (
None if query is None else quote(query, safe=SUB_DELIMS + ":/?[]@")
)
Expand Down