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

Disable uploading on ESR #3996

Closed
ghost opened this issue Jan 22, 2018 · 4 comments
Closed

Disable uploading on ESR #3996

ghost opened this issue Jan 22, 2018 · 4 comments
Assignees

Comments

@ghost
Copy link

ghost commented Jan 22, 2018

Firefox 60 is an ESR. To avoid problems down the line, we'd like to disable the ability to save screenshots to our servers on ESR releases (note: just ESR, not plain Firefox 60). The screenshots functionality will still be there, but the Save button will be omitted.

From our discussion:

  • We should add a pref to do this and convert our current disable-save-button logic to use it
  • Add ESR detection to flip the pref
  • Add the pref to our documentation and/or SUMO
@ghost ghost added this to the Sprint 5 (60-0) 🔘 milestone Jan 22, 2018
@jaredhirsch
Copy link
Member

We can detect ESR-hood via AppConstants.MOZ_UPDATE_CHANNEL == 'esr'. No need to set a pref.

@jaredhirsch jaredhirsch self-assigned this Jan 22, 2018
@ianb
Copy link
Contributor

ianb commented Jan 22, 2018

It feels like the pref is generally useful (especially given the enterprise policy stuff that's being deployed). The reason given that some people/admins want to disable Screenshots is specifically to disable uploading of data (e.g., for computers subject to HIPAA).

@jaredhirsch
Copy link
Member

Are you thinking of a separate pref to just disable uploads, like extensions.screenshots.upload-disabled? If so, maybe that could be a separate bug.

I'm proposing that uploads always be disabled if a Firefox is on the ESR channel, which seems like the goal of this bug.

@jaredhirsch
Copy link
Member

Ah, the pref is mentioned explicitly in the description for this bug, so I'll just add it 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants