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

Status: RESOLVED FIXED

(Core :: Networking, task, P3)

Description

[razielim]

Steps to reproduce:

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

See: https://github.com/whatwg/url/pull/685

Firefox is already mostly compliant with this change (it doesn't appear to support hostnames for non-special URLs yet, so at least that part of the change doesn't lead to a new regression). However, it currently blocks %7F in domains but allows raw U+007F. It should disallow both forms.

See the regression at: https://wpt.fyi/results/url/url-constructor.any.html?run_id=5754969316655104&run_id=5720618772201472

Comment 1

[razielim]

Apologies for the typo - "Opaque hostnames remain changed." should be "Opaque hostnames remain unchanged." 🤦‍♂️

Comment 2

[Kershaw Chang [:kershaw]]

Hi Valentin,

What do you think about the priority of this one?
Should we create a meta bug for this kind of issue?

Thanks.

Comment 3

[Kershaw Chang [:kershaw]]

Tom, thanks for the meta bug.

Comment 4

[Valentin Gosu [:valentin] (he/him)]

Thanks!

Comment 5

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1752320 - Add U+007F to the invalid host characters list r=#necko

Tests will be merged from upstream WPT:
https://github.com/web-platform-tests/wpt/pull/32524

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/656e7e2fe5c2
Add U+007F to the invalid host characters list r=necko-reviewers,kershaw

Comment 7

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/656e7e2fe5c2