1895233 - Potentially incorrect parsing of <file:w|m>

Status: RESOLVED FIXED

(Core :: Networking, defect, P3)

Description

[Tom Schuster]

We seem to parse file:w|m as "file:///w:m" instead of "file:///w|m" as expected by whatwg-url at least.

Comment 1

[Valentin Gosu [:valentin]]

file:C|/m/ should parse as file:///C:/m/, so I think the important bit here is the / after the |

Comment 2

[Sean Kim [:sekim]]

Attachment: Bug 1895233 - Fix incorrect parsing of <file:w|m> by detecting file:|/ in BuildNormalizedSpec r?valentin

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/87d94a0e7911
Fix incorrect parsing of <file:w|m> by detecting file:|/ in BuildNormalizedSpec r=valentin,necko-reviewers

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/46720 for changes under testing/web-platform/tests

Comment 5

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/87d94a0e7911

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot