Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

WebExtensions in Private Browsing #6123

Closed
vesta0 opened this issue Oct 18, 2019 · 6 comments
Closed

WebExtensions in Private Browsing #6123

vesta0 opened this issue Oct 18, 2019 · 6 comments

Comments

@vesta0
Copy link
Collaborator

vesta0 commented Oct 18, 2019

User Story

As a user, I want to choose whether I want to allow a WebExtensions in private browsing, so I have control over the data and permissions shard in private browsing.

Dependencies

A-C project board

Acceptance Criteria

  • I can choose to allow a WebExtension to run in private browsing
  • This option is at the individual WebExtension level

┆Issue is synchronized with this Jira Task

@vesta0 vesta0 added this to Q1 User Stories in A-C: WebExtensions and AddOns Oct 18, 2019
@vesta0 vesta0 added this to Prioritized UX Backlog in Fenix Sprint Kanban Oct 21, 2019
@brampitoyo brampitoyo moved this from Prioritized UX Backlog to In Design in Fenix Sprint Kanban Nov 4, 2019
@brampitoyo brampitoyo added the ux:s label Nov 5, 2019
@brampitoyo
Copy link

Progress report:

  • In Q4 2019, we’ve decided that WebExtensions will run in both normal and private browsing modes
  • In the future, we may provide a choice to disable individual WebExtension in private browsing

@vesta0 vesta0 removed this from In Design in Fenix Sprint Kanban Dec 10, 2019
@csadilek csadilek added the needs:gv GeckoView bug required to fix the issue. See bugzilla.mozilla.org label Feb 19, 2020
@csadilek
Copy link
Contributor

@csadilek csadilek added needs:ac Needs Android Component Work and removed needs:gv GeckoView bug required to fix the issue. See bugzilla.mozilla.org labels Mar 10, 2020
@st3fan st3fan moved this from ⏳ Sprint Backlog to 🏃‍♀️ In Progress in A-C: Android Components Sprint Planning Apr 13, 2020
psymoon added a commit to psymoon/fenix that referenced this issue Apr 21, 2020
psymoon added a commit to psymoon/fenix that referenced this issue Apr 21, 2020
psymoon added a commit to psymoon/fenix that referenced this issue Apr 23, 2020
psymoon added a commit to psymoon/fenix that referenced this issue Apr 23, 2020
@psymoon psymoon added eng:qa:needed QA Needed and removed needs:ac Needs Android Component Work labels Apr 23, 2020
@csadilek
Copy link
Contributor

@brampitoyo As the default for this is "false" (add-ons do not run in private browsing until users turn it off) do we need some form of hint / notification? This is re: #10120 (comment).

Defaulting to false seems correct as this is what Desktop does, just wondering if we need to make this more obvious to users?

@brampitoyo
Copy link

@csadilek If these are true:

  • Many popular add-ons – and most add-ons that Fenix supports – are ad blockers and/or focused around security
  • Private browsing typically demands more stringent security, stricter trackers blocking, etc.

Then does it make sense to set our default policy to TRUE: allow add-on to run in private browsing?

I immediately see huge problems around consent. Add-ons may “phone home” and leak private data (e.g. URLs opened in private browsing). But does our Recommended Extension programme not protect against this?

I think that many users’ expectation is for add-ons to work everywhere by default, but also for those add-ons to not leak data while browsing privately.


Either way, I do agree that WebExtensions that run in private browsing should be clearly labelled.

I propose doing something similar to desktop: have a small (16×16dp) private browsing icon to the side of the add-on name. Thoughts?

Icon asset: https://dzwonsemrish7.cloudfront.net/items/2D2h3z2m2P3G0Q2a3I0y/private-browsing.svg

@AndiAJ
Copy link
Collaborator

AndiAJ commented Apr 27, 2020

Hi, verified as fixed on the latest Nightly Build #21180609 from 4/27 using the following devices:
• Google Pixel 3a (Android 10)
• Huawei Mate 20 Lite (Android 9)

✔️ I can choose to allow a WebExtension to run in private browsing
✔️ This option is at the individual WebExtension level

► Video
20200427-162842

@AndiAJ AndiAJ closed this as completed Apr 27, 2020
A-C: Android Components Sprint Planning automation moved this from 🏃‍♀️ In Progress to 🏁 Done Apr 27, 2020
@AndiAJ AndiAJ added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Apr 27, 2020
@csadilek
Copy link
Contributor

@brampitoyo Thanks!

I immediately see huge problems around consent. Add-ons may “phone home” and leak private data (e.g. URLs opened in private browsing). But does our Recommended Extension programme not protect against this?

Agreed. I think defaulting to "false" is the right thing to do here, so users can opt-in, as desired. Having a list of recommended and tested extensions protects us from this somewhat, but it still could become a problem later. So, I'd vote for being conservative and use the same default as Desktop.

I will file a separate issue to introduce the label you provided above.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
A-C: WebExtensions and AddOns
  
Q2 User Stories
Development

No branches or pull requests

5 participants