1864939 - URL parsing of ssh:// adds a trailing slash to the href

Status: RESOLVED DUPLICATE of bug 1326394

(Core :: Networking, defect)

Description

[[e-mail]]

Attachment: firefox-bug.png

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/110.0

Steps to reproduce:

In a dumb page with a link to a new protocol like ssh://10.10.23.136
Firefox detects it correctly and offers me an SSH application.
Application is receiving to open 10.10.23.136/

Actual results:

Application is receiving to open 10.10.23.136/
an the SSH application can not resolve 10.10.23.136/

Expected results:

The application must receive exactly the same content as that encoded in the HTML. No extra characters

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Core::Widget: Gtk' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 2

[Andrew McCreight [:mccr8]]

Gijs, is this likely to be an external protocol handler issue? That's kind of what it feels like to me. Thanks.

Comment 3

[:Gijs (he/him)]

(In reply to Andrew McCreight [:mccr8] from comment #2)

Gijs, is this likely to be an external protocol handler issue? That's kind of what it feels like to me. Thanks.

No, this is a DOM issue, because:

new URL("ssh://12.12.12.12").href

has the trailing slash. Valentin would know why; I would have assumed a spec requirement, but testing in chrome it does not get the trailing slash. Anyway, by the time the external protocol handler sees it, there's no way to find the original attribute value (different process etc.).

Comment 4

[Andrew McCreight [:mccr8]]

Thanks. I think we put URL parsing in Networking now so I'll move it there.

It looks like Chrome DOES add the trailing slash for https://, http:// and ftp:// but does NOT for other miscellaneous protocols like blob://, gopher:// and gijs://.

Comment 5

[Andrew McCreight [:mccr8]]

Firefox DOES add the trailing slash for https://, but does NOT for blob://, gopher:// and gijs://. So in my limited testing it is only ssh:// where we disagree.

Comment 6

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

Duplicate of bug 1326394.
Will be fixed by bug 1603699.