Unrestrict eval telemetry on beta
Categories
(Core :: DOM: Security, task, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox69 | --- | unaffected |
firefox70 | - | fixed |
firefox71 | --- | unaffected |
People
(Reporter: tjr, Assigned: tjr)
References
(Blocks 1 open bug)
Details
(Whiteboard: [domsecurity-active])
Attachments
(1 obsolete file)
The intent here was to collect telemetry from Beta 70 (with no enforcement) and then Nightly 71 would enforce it.
But I forgot we had Nightly guards on the function call so we're not calling it on Beta or sending in Telemetry.
So I'm asking for Beta uplift here to remove the Nightly guards.
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
[Tracking Requested - why for this release]: The intent here is to remove Nightly guards from something we want to call in Beta
Updated•5 years ago
|
Assignee | ||
Comment 3•5 years ago
|
||
Comment on attachment 9091124 [details]
Bug 1579495 - Remove Nightly guards on eval telemetry r?ckerschb
Beta/Release Uplift Approval Request
- User impact if declined: We'll have to wait another cycle until we can begin enforcement of this security feature.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Medium
- Why is the change risky/not risky? (and alternatives if risky): We'll calling a new codepath that submits telemetry events. It's possible an unexpected environment difference between beta and Nightly could lead to getting flooded with telemetry.
- String changes made/needed:
Comment on attachment 9091124 [details]
Bug 1579495 - Remove Nightly guards on eval telemetry r?ckerschb
Enabling this telemetry in beta 6.
Please keep an eye on whether we're flooded with telemetry or not on Friday when we release beta 6.
Comment 6•5 years ago
|
||
bugherder uplift |
Assignee | ||
Comment 7•5 years ago
|
||
This isn't landing in central, so it's fixed, but leaving the ni open for me to confirm about Telemetry.
Updated•5 years ago
|
Description
•