Closed
Bug 1942328
Opened 1 year ago
Closed 1 year ago
Add www.firefox.com to UITour allow-list permissions
Categories
(Firefox :: Tours, enhancement)
Firefox
Tours
Tracking
()
RESOLVED
FIXED
136 Branch
People
(Reporter: agibson, Assigned: willdurand)
References
Details
(Whiteboard: [addons-jira])
Attachments
(2 files)
|
48 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-esr115+
pascalc
:
approval-mozilla-esr128+
|
Details | Review |
|
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-esr115+
|
Details | Review |
The websites team is currently preparing to stand up a new website at https://www.firefox.com. We would like this website to have the same UITour privileges as www.mozilla.org, and as such we'd like to add https://www.firefox.com to the allow-list for UITour permitted websites.
https://searchfox.org/mozilla-central/source/browser/app/permissions
| Assignee | ||
Comment 1•1 year ago
|
||
Updated•1 year ago
|
Assignee: nobody → wdurand
Status: NEW → ASSIGNED
| Assignee | ||
Updated•1 year ago
|
Whiteboard: [addons-jira]
Updated•1 year ago
|
Pushed by wdurand@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/604420788099
Add www.firefox.com to UITour allow-list permissions. r=emz
Comment 3•1 year ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
| Assignee | ||
Comment 4•8 months ago
|
||
Comment on attachment 9461342 [details]
Bug 1942328 - Add www.firefox.com to UITour allow-list permissions. r?emz!
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: www.firefox.com has some Firefox-specific features enabled by the
uitourpermission. Old ESR versions don't have that permission, hence why we'd like to uplift this patch to ESR 115 and 128. - User impact if declined: Users of ESR 115 and 128 browsing www.firefox.com.
- Fix Landed on Version: 136
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): It's not risky because the patch is minimal and self-contained.
Attachment #9461342 -
Flags: approval-mozilla-esr128?
Attachment #9461342 -
Flags: approval-mozilla-esr115?
Updated•8 months ago
|
Attachment #9461342 -
Flags: approval-mozilla-esr128?
Attachment #9461342 -
Flags: approval-mozilla-esr128+
Attachment #9461342 -
Flags: approval-mozilla-esr115?
Attachment #9461342 -
Flags: approval-mozilla-esr115+
Updated•8 months ago
|
status-firefox-esr128:
--- → fixed
tracking-firefox-esr128:
--- → 141+
Comment 6•8 months ago
|
||
William, could you provide an updated patch for esr115? The one in this bug does not apply cleanly to the branch, thanks.
Flags: needinfo?(wdurand)
| Assignee | ||
Comment 7•8 months ago
|
||
Updated•8 months ago
|
Attachment #9498815 -
Flags: approval-mozilla-esr115?
| Assignee | ||
Comment 8•8 months ago
|
||
:pascalc I attached a new patch for ESR115, thanks.
Flags: needinfo?(wdurand)
Updated•8 months ago
|
Attachment #9498815 -
Flags: approval-mozilla-esr115? → approval-mozilla-esr115+
Updated•8 months ago
|
status-firefox-esr115:
--- → affected
status-firefox-esr140:
--- → fixed
tracking-firefox-esr115:
--- → 141+
Updated•8 months ago
|
Updated•8 months ago
|
QA Whiteboard: [qa-triage-done-c142/b141]
You need to log in
before you can comment on or make changes to this bug.
Description
•