Add FTP related protocols to the registerProtocolHandler safelist.
Categories
(Core :: Networking, enhancement, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox98 | --- | fixed |
People
(Reporter: asanka, Assigned: valentin)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete, Whiteboard: [necko-triaged])
Attachments
(2 files)
As a part of removing FTP support in Chromium (and in turn Google Chrome, Edge, etc.) I'd like to release ftp, sftp, ftps protocols as being registrable via registerProtocolHandler.
ChromeStatus entry is here: https://www.chromestatus.com/feature/5762628536238080
I'll follow up with relevant PR to whatwg/html and wpt.
Blink I2S thread is https://groups.google.com/a/chromium.org/g/blink-dev/c/ABhlioapE0E/m/gXL84wPfAAAJ . Please express your support objections here or there as appropriate.
|
Reporter | |
Comment 1•4 years ago
|
||
Related whatwg/html issue : https://github.com/whatwg/html/issues/6583
|
Assignee | |
Comment 2•4 years ago
|
||
Hi Asanka. You should file an issue on https://github.com/mozilla/standards-positions/issues to get a position on this topic.
I'll keep this bug for necessary changes if we reach a decision.
|
|
Reporter | |
Comment 3•4 years ago
|
||
Thank! Did so. https://github.com/mozilla/standards-positions/issues/513
|
||
Comment 4•4 years ago
|
||
Ah, I saw this after the GitHub issue. 😊 This seems reasonable to me now FTP is no longer a scheme the browser handles.
|
|
Assignee | |
Updated•3 years ago
|
|
||
Comment 5•3 years ago
|
||
FYI: The protocol_handlers extension API supports ftp since bug 1626365 (https://hg.mozilla.org/mozilla-central/rev/bcdf1f609632#l3.2).
|
|
Assignee | |
Comment 6•3 years ago
|
||
|
|
Assignee | |
Comment 7•3 years ago
|
||
According to https://github.com/mozilla/standards-positions/issues/513
the safe thing to do for FTP URLs when using a protocol handler is to strip
the credentials from them first.
This also adds a test that makes sure credentials are stripped when using
a web protocol handler.
Depends on D136121
|
|
Assignee | |
Updated•3 years ago
|
|
||
Comment 9•3 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/b52cf82fbe03
https://hg.mozilla.org/mozilla-central/rev/882c7f9567cd
|
||
Comment 10•3 years ago
|
||
FF98 docs work for this can be tracked in https://github.com/mdn/content/pull/12790
I see from https://github.com/whatwg/html/issues/6583#issue-858243907 that this change is currently non-spec. Will this bugfix remain in the release even if the PR is not accepted?
This will affect how the browser compatibility and MDN documents are updated.
|
|
||
Updated•3 years ago
|
|
|
Assignee | |
Comment 11•3 years ago
|
||
Presumably this change will be backed out if the PR gets rejected, but that's unlikely given this is a widely supported change.
|
|
||
Comment 12•3 years ago
|
||
Thanks! The spec got updated. I have marked as dev-doc-complete. The browser compatibility update is still in discussion though (you can follow that through issue linked in https://github.com/mdn/content/pull/12790)
Description
•