Right now, MatchPattern permits ftp://, but not FTP over TLS (ftps://) URLs, which is in contrast to http:// also permitting HTTP over TLS (https://). This bug serves to track the work necessary to add ftps:// to the list of valid WebExtension MatchPattern schemes.

The list of allowed schemes is here: https://searchfox.org/mozilla-central/source/toolkit/components/extensions/MatchPattern.cpp#266 if you're interested in contributing a patch :)

Mentor: Shane Caraveo (:mixedpuppy) If this is your first contribution, please refer to https://wiki.mozilla.org/Add-ons/Contribute/Code on how to get started.

Given comments in bug 85464, closing.

Comment on attachment 8998394 [details] Bug 1463440 - Add FTP over TLS/SSL (ftps://) URLs to MatchPattern Shane Caraveo (:mixedpuppy) has approved the revision.

Re‑opening, as :mixedpuppy approved this.

Does Firefox actually support FTPS ? If not, there's no point adding this to WebExtensions.

(In reply to Tim Nguyen :ntim from comment #7) > Does Firefox actually support FTPS ? Implementation of FTPS is tracked in bug 85464, which :ckerschb closed in bug 85464 comment 35 ☹️ > If not, there's no point adding this to WebExtensions. As I said in the patch on Phabricator (https://phabricator.services.mozilla.com/D2899#65206), this will at least allow WebExtensions to use the `browser.webRequest` API to intercept FTPS calls. Also, even if this doesn’t end up actually doing anything, at least we won’t forget to add it to MatchPattern as part of bug 85464.

OK, it should probably be documented then.

(In reply to Tim Nguyen :ntim from comment #9) > OK, it should probably be documented then. Way ahead of you (all done within a week of opening this bug): - https://github.com/mdn/browser-compat-data/commit/b8c7e1b9133d6dafc479fd140ba55ae3b0bf85d2 - https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/Match_patterns#scheme - https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/Match_patterns#Examples - https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/Match_patterns#Browser_compatibility

(In reply to ExE Boss from comment #8) > (In reply to Tim Nguyen :ntim from comment #7) > > Does Firefox actually support FTPS ? > > Implementation of FTPS is tracked in bug 85464, which :ckerschb closed in > bug 85464 comment 35 ☹️ > > > If not, there's no point adding this to WebExtensions. > > As I said in the patch on Phabricator > (https://phabricator.services.mozilla.com/D2899#65206), this will at least > allow WebExtensions to use the `browser.webRequest` API to intercept FTPS > calls. > > Also, even if this doesn’t end up actually doing anything, at least we won’t > forget to add it to MatchPattern as part of bug 85464. There goes my lack of proper cafination in the morning. Since we don't actually support ftps then we probably shouldn't add it to matchpattern in a way that makes it potentially work with permissions in a manifest. So basically, sorry, lets hold off unless there is an actual working use case.

Comment on attachment 8998394 [details] Bug 1463440 - Add FTP over TLS/SSL (ftps://) URLs to MatchPattern removing r+

Going back to comment 3 and closing again.