Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Audit uses of urllib.parse.quote to see if the string can contain forward slashes #14511

Open
clokep opened this issue Nov 21, 2022 · 0 comments
Open

Audit uses of urllib.parse.quote to see if the string can contain forward slashes #14511

clokep opened this issue Nov 21, 2022 · 0 comments
Labels
T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. Z-Cleanup Things we want to get rid of, but aren't actively causing pain

Comments

@clokep
Copy link
Member

clokep commented Nov 21, 2022

As @squahtx says, we should audit other uses of urllib.parse.quote to see if there are other problems like this.

Grep for ((urllib\.)?parse\.)?quote\(.

Originally posted by @DMRobertson in #14488 (comment)
@clokep clokep mentioned this issue Nov 21, 2022
Closed
@DMRobertson DMRobertson added T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. Z-Cleanup Things we want to get rid of, but aren't actively causing pain labels Nov 22, 2022
@matrixbot matrixbot mentioned this issue Dec 21, 2023
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. Z-Cleanup Things we want to get rid of, but aren't actively causing pain
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@clokep @DMRobertson