1873944 - Properly parse data and about URLs that have a hostname

Status: RESOLVED FIXED

(Core :: Networking, defect, P3)

Description

[Valentin Gosu [:valentin]]

data and about URLs are parsed with nsSimpleURI currently.
that means everything after the first : is part of the path. But these schemes are not special (in the URL spec sense), so they could potentially have a host.

This bug tracks handling these corner cases in WPT, even though they would normally not affect developers.

Comment 1

[Valentin Gosu [:valentin]]

What we could do is if the URL starts with data:/ or about:/ then we first parse it with DefaultURI and then use the result to construct a nsSimpleURI. 🤷‍♂️

Comment 2

[Dylan]

Attachment: Bug 1873944 - Also parse about and data urls with DefaultURI if starting with '/'. r=valentin

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c6777d2ebff8
Also parse about and data urls with DefaultURI if starting with '/'. r=necko-reviewers,valentin,rpl

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/c6777d2ebff8

Comment 5

[:Gijs (he/him)]

Per discussion with Valentin, I think we want to back this out if that is still possible, given some regressions.

Comment 6

[Valentin Gosu [:valentin]]

I filed bug 1997895 to be able to track affected builds more easily.