-
Notifications
You must be signed in to change notification settings - Fork 264
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
HOST accepts comma chars #198
Comments
Yup, according to the RFC (but probably not DNS), that's correct. Addressable does not currently consider DNS in validating a URI, something that was a core motivation for writing Addressable in the first place. Per the comment on twingly/twingly-uri#74, I'm considering a |
Just to connect the issues (because of the typo above), here is a link to the comment: twingly/twingly-url#74 (comment) |
This handling was updated in f032606, but since commas are in |
I noticed that
Addressable::URI.parse
do not take'http://www,google.com'
as invalid, behavior which I was expecting.Is that correct?
Thanks in advance!
The text was updated successfully, but these errors were encountered: