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

[URL] Split forbidden host/domain code points, and add all C0 controls and add U+007F to the latter #41717

Closed
karwa opened this issue Jan 27, 2022 · 1 comment · Fixed by #43190
Labels
whatwg-url Issues and PRs related to the WHATWG URL implementation.

Comments

@karwa
Copy link

karwa commented Jan 27, 2022

Version

N/A

Platform

N/A

Subsystem

URL

What steps will reproduce the bug?

No response

How often does it reproduce? Is there a required condition?

No response

What is the expected behavior?

Spec update: Domains may not contain C0 controls or U+007F DELETE. Opaque hostnames remain unchanged.

See: whatwg/url#685

What do you see instead?

N/A

Additional information

Sorry if this is not the right way to submit this issue. It was either this (bug) or a feature request. And it didn't seem like a feature request.

@targos targos added the whatwg-url Issues and PRs related to the WHATWG URL implementation. label Jan 27, 2022
@benjamingr
Copy link
Member

@nodejs/url note this landed so to align with the spec it should probably be fixed in Node

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
whatwg-url Issues and PRs related to the WHATWG URL implementation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants