Open Bug 1299571 Opened 8 years ago Updated 2 years ago

Can remove about:addons

Categories

(WebExtensions :: General, defect, P3)

defect

Tracking

(Not tracked)

REOPENED

People

(Reporter: andy+bugzilla, Unassigned)

References

Details

(Keywords: sec-low, Whiteboard: [triaged])

Attachments

(1 file)

607 bytes, application/x-xpinstall
Details
Attached file close-about-addons.xpi
We've seen add-ons that do malicious things like blank out about:addons. The intention is to not allow a user to spot the malicious add-on and uninstall it. 

If that about:addons is special, then WebExtensions can similar remove access to about:addons by just closing the tab.

Attached example.
On the upside, users can always start Firefox in safe mode to fix this.

It would definitely be nice to prevent add-ons from doing something like this maliciously, but I'm not sure that we could do it without significantly handicapping tab add-ons, and the like. But maybe we can restrict them so that they can only navigate away from pages like about:addons in response to user input of some sort.
Keywords: sec-low
Priority: -- → P5
Whiteboard: [triaged]
I don't think there's much we should do here. WebExtensions aren't perfect, but we also don't want to make them too complicated with these sorts of changes.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Group: toolkit-core-security
I don't know that there's much we should do, but doing nothing leaves a problem open (such as what a block was used for in https://bugzilla.mozilla.org/show_bug.cgi?id=1454691#c0). #c1 makes sense to me, seems like we should see if we can prevent the obvious cases that would befuddle users.

Maybe we should discuss whether or not there's a known subset of things we can guard against, or whether our approach for these myriad kinds of problems should rely on blocklisting.
Status: RESOLVED → REOPENED
Priority: P5 → P3
Resolution: WONTFIX → ---
See Also: → 1455360
If the long term plan is to remove about:addons and move the logic over to about:preferences, perhaps focusing on that page might be the way to go. Since about:preferences is used to show the impact of extensions, that might be a legitimate target anyway.

One other thought I had in relation to moving out of about:addons is moving the add-on data to a location which can not be affected by add-ons at all. Unlike a tab. For example, the jigsaw button in the menu bar could be turned into an add-on listing and WebExtensions cannot remove that.
Any links you can provide for that long-term plan would certainly be helpful for the increasing questions about it. :)
Flags: needinfo?(andy)
If I remember correctly, that's project Medley which Emanuela was working on, they might be able to help.
Flags: needinfo?(andy) → needinfo?(emanuela)
Product: Toolkit → WebExtensions
Yes, there are still plans to leave about:addons *bye bye* and move the managament of extensions in about:preferences. 
Unfortunatly, I cannot provide a proper timeline for this. Maybe David can help us.
Flags: needinfo?(emanuela) → needinfo?(ddurst)
Prioritization would happen after we have a clear plan, and signoff from the team that owns about:preferences.
Flags: needinfo?(ddurst)
Similar to Comment 6 by Andy (new to Bugzilla ;-) we talked about having a "Disable Extensions" menu item, or even listing of extensions, as a submenu to the puzzle piece.  There are a lot of ways to do this, but the idea is the same - have a definitive place users can go where they can disable/manage extensions, which means it needs to be "above the fold" and part of the browser chrome.
My first thought was to just shut off any notifications to extensions regarding about urls (e.g. tabs.onUpdated).  That of course will break some legit use.  To fix that, we can introduce a systemTabs permission.  Any extension that wants to get the events will require that permission.  At least in that way, we would have an identifier.
Bulk move of bugs per https://bugzilla.mozilla.org/show_bug.cgi?id=1483958
Component: Untriaged → General
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: