Skip to content
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

Forbid | in URL hosts #589

Merged
merged 1 commit into from
Mar 22, 2021
Merged

Forbid | in URL hosts #589

merged 1 commit into from
Mar 22, 2021

Conversation

achristensen07
Copy link
Collaborator

@achristensen07 achristensen07 commented Mar 17, 2021

This fixes issue #559.

(See WHATWG Working Mode: Changes for more details.)


Preview | Diff

@annevk annevk merged commit 4025253 into whatwg:main Mar 22, 2021
annevk pushed a commit to web-platform-tests/wpt that referenced this pull request Mar 22, 2021
@annevk
Copy link
Member

annevk commented Mar 22, 2021

Thanks @achristensen07! I filed Chrome and Firefox bugs. I assume Safari doesn't need one.

@achristensen07
Copy link
Collaborator Author

Thanks for filing those bugs. Safari has already made this change.

@jasnell
Copy link
Collaborator

jasnell commented Mar 22, 2021

Bug report for Node.js: nodejs/node#37862

domenic added a commit to jsdom/whatwg-url that referenced this pull request Mar 24, 2021
domenic added a commit to jsdom/whatwg-url that referenced this pull request Mar 24, 2021
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Mar 27, 2021
Automatic update from web-platform-tests
URL: forbid | in hosts

Tests for whatwg/url#589.
--

wpt-commits: 5eebfdb1f68059549b3efff380dd190bc6078266
wpt-pr: 28118
valenting added a commit to servo/rust-url that referenced this pull request Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants