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 #13509

Closed
karwa opened this issue Jan 27, 2022 · 4 comments
Assignees
Labels
bug Something isn't working upstream Changes in upstream are required to solve these issues web related to Web APIs

Comments

@karwa
Copy link

karwa commented Jan 27, 2022

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

See: whatwg/url#685

@lucacasonato lucacasonato added upstream Changes in upstream are required to solve these issues web related to Web APIs labels Jan 27, 2022
@lucacasonato
Copy link
Member

Upstream change in https://github.com/servo/rust-url is required. @crowlKats will give it a shot.

@stale

This comment was marked as outdated.

@stale stale bot added the stale label Mar 29, 2022
@crowlKats crowlKats removed the stale label Mar 29, 2022
@stale

This comment was marked as outdated.

@stale stale bot added the stale label May 31, 2022
@stale stale bot closed this as completed Jun 11, 2022
@lucacasonato lucacasonato added the bug Something isn't working label Jun 11, 2022
@lucacasonato lucacasonato reopened this Jun 11, 2022
@stale stale bot removed the stale label Jun 11, 2022
@crowlKats
Copy link
Member

This has since been fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working upstream Changes in upstream are required to solve these issues web related to Web APIs
Projects
None yet
Development

No branches or pull requests

3 participants