-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[UrlValidator] Url with curly braces is not correctly validated #58720
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.
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
Comments
I am not good in RFC's =) But after a quick search, I found nothing regarding curly braces. Moreover, square brackets are in the list of reserved characters, but they are allowed in regular expressions and browsers processed as well. curl "https://www.google.com/search?q=%7Bprice%7D" -o search1.html Gives curl "https://www.google.com/search?q={price}" -o search2.html Gives Oh, find more interesting) curl "https://www.google.com/search?q=[price]" -o search5.html
curl: (3) bad range in URL position 34:
https://www.google.com/search?q=[price]
^ But curl -g "https://www.google.com/search?q=[price]" -o search5.html works well and pass brackets to input, the same with curly braces. |
Hey, thanks for your report! |
Friendly reminder that this issue exists. If I don't hear anything I'll close this. |
Be careful that the
That comment looks weird to me. You were testing with |
Symfony version(s) affected
v7.1.6
Description
Found that correct url processed with error by url validator
Seems we need to add curly braces to allowed symbols in regex for query section of url.
How to reproduce
Possible Solution
current
added
\{\}
Additional Context
No response
The text was updated successfully, but these errors were encountered: