-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
URL: test host parsing of URLs with unknown schemes #4406
Conversation
Notifying @Sebmaster, @domenic, @frewsxcv, @mikewest, @rubys, @sideshowbarker, @smola, @tomalec, @xiaojunwu, and @zcorpan. (Learn how reviewing works.) |
47efa6b
to
7d81096
Compare
For URLs without a special scheme we cannot use the host parser directly due to compatibility issues. Instead we percent-encode the input. Tests: web-platform-tests/wpt#4406. Fixes #148.
19ee073
to
5406bb0
Compare
5406bb0
to
1d52991
Compare
These used to be part of #4406.
Firefox (nightly channel)Testing web-platform-tests at revision 045b749 |
Chrome (unstable channel)Testing web-platform-tests at revision 290faf8 All results/url/url-constructor.html
/url/url-setters.html
/url/url-origin.html
/url/a-element-xhtml.xhtml
/url/a-element.html
/url/a-element-origin.html
/url/a-element-origin-xhtml.xhtml
|
For URLs without a special scheme we cannot use the host parser directly due to compatibility issues. Instead we percent-encode the input. Also make sure that if "userinfo" or port is present, host is non-empty. Tests: web-platform-tests/wpt#4406. Fixes #148 and fixes #214.
These used to be part of #4406.
These used to be part of #4406.
Some of these used to be part of #4406. See also whatwg/url#212.
URL Standard change: whatwg/url#185.
This change is