FTP removal and testing with addons
Categories
(WebExtensions :: General, task, P2)
Tracking
(firefox88 fixed)
Tracking | Status | |
---|---|---|
firefox88 | --- | fixed |
People
(Reporter: mixedpuppy, Assigned: mixedpuppy)
References
(Blocks 2 open bugs, )
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
This meta is intended to block disabling FTP being turned off beyond Nightly until we have resolved any questions or issues resulting from the removal. The google doc listed in the url field is a WIP document to outline any potential issues and questions.
Specifically, FTP is being disabled on Nightly in Bug 1622409
Assignee | ||
Comment 1•1 year ago
|
||
Once I organize my thoughts better and verify some things we'll make the document public.
Assignee | ||
Comment 2•1 year ago
|
||
Some useful notes that may end up as other bugs:
FTP: there are some other mentions in the documentation:
browserSettings.ftpProtocolEnabled - will be removed or deprecated?
Proxy.settings as an FTP property - will be removed or deprecated?
tabs.executeScript() notes that you can inject code from an FTP source - Remove?
proxy.register() (deprecated) notes that it will be called for any FTP request - Remove?
tabs.insertCSS() notes that the CSS can be from an FTP source - Remove?
Work with the Bookmarks API notes that you can bookmark an FTP site - will still be valid?
events.UrlFilter uses an FTP address is an example of matched URLs (urlMatches) - removed to avoid confusion?
Assignee | ||
Comment 3•2 months ago
|
||
should allow ftp in protocol handlers when this happens.
Assignee | ||
Comment 4•1 month ago
|
||
Remove the ability for extensions to toggle ftp support on, and add protocol handler support.
Updated•1 month ago
|
Assignee | ||
Updated•1 month ago
|
Assignee | ||
Updated•1 month ago
|
Pushed by scaraveo@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bcdf1f609632 better extension support for ftp removal r=robwu
Comment 8•1 month ago
|
||
bugherder |
Comment 9•22 days ago
|
||
Hi Shane,
I've started docs for this in on MDN content and Browser compatibility.
Currently this covers making browserSettings.ftpProtocolEnabled
readonly.
This also mentions that you added "better extension support". What does that mean for an extension developer? What else can they do/where are you expecting docs?
In https://bugzilla.mozilla.org/show_bug.cgi?id=1626365#c2 you mention all these things which might change - did they, and if so how? For example can we still bookmark an FTP URL. If nothing has changed I'll leave the docs alone.
browserSettings.ftpProtocolEnabled - will be removed or deprecated?
Proxy.settings as an FTP property - will be removed or deprecated?
tabs.executeScript() notes that you can inject code from an FTP source - Remove?
proxy.register() (deprecated) notes that it will be called for any FTP request - Remove?
tabs.insertCSS() notes that the CSS can be from an FTP source - Remove?
Work with the Bookmarks API notes that you can bookmark an FTP site - will still be valid?
events.UrlFilter uses an FTP address is an example of matched URLs (urlMatches) - removed to avoid confusion?
Did any of them?
Assignee | ||
Comment 10•19 days ago
|
||
(In reply to Hamish Willee from comment #9)
Hi Shane,
I've started docs for this in on MDN content and Browser compatibility.
Currently this covers makingbrowserSettings.ftpProtocolEnabled
readonly.This also mentions that you added "better extension support". What does that mean for an extension developer? What else can they do/where are you expecting docs?
"ftp" has been added to protocol_handlers, see list for protocol
at https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/protocol_handlers
In https://bugzilla.mozilla.org/show_bug.cgi?id=1626365#c2 you mention all these things which might change - did they, and if so how? For example can we still bookmark an FTP URL. If nothing has changed I'll leave the docs alone.
It may be able to be bookmarked, but would no longer load in Firefox.
browserSettings.ftpProtocolEnabled - will be removed or deprecated?
Setting the value is removed, but addons can still read the value.
Proxy.settings as an FTP property - will be removed or deprecated?
tabs.executeScript() notes that you can inject code from an FTP source - Remove?
proxy.register() (deprecated) notes that it will be called for any FTP request - Remove?
tabs.insertCSS() notes that the CSS can be from an FTP source - Remove?
Work with the Bookmarks API notes that you can bookmark an FTP site - will still be valid?
events.UrlFilter uses an FTP address is an example of matched URLs (urlMatches) - removed to avoid confusion?
Using ftp with any of these will no longer have any effect.
Comment 11•14 days ago
|
||
Thanks Shane. I've fixed these up (mostly by removing the references) in https://github.com/mdn/content/pull/3868
Comment 12•5 days ago
|
||
Docs complete for this change; see https://github.com/mdn/content/issues/3462 for the discussion.
Description
•