Closed
Bug 1494589
Opened 6 years ago
Closed 6 years ago
Extend PERMISSION_REQUEST_* probe lifetime
Categories
(Firefox :: Site Identity, enhancement, P1)
Firefox
Site Identity
Tracking
()
RESOLVED
FIXED
Firefox 64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: johannh, Assigned: johannh)
References
Details
Attachments
(2 files)
We have a bunch of work in progress that benefits from or needs these measurements (bug 1483631, bug 1429432), so it makes sense to keep these around a little longer.
Assignee | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Comment on attachment 9012521 [details]
Bug 1494589 - Extend PERMISSION_REQUEST_* probe lifetime. r=florian,data-r=chutten
Florian Quèze [:florian] has approved the revision.
Attachment #9012521 -
Flags: review+
Pushed by jhofmann@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/55be882db711
Extend PERMISSION_REQUEST_* probe lifetime. r=florian
Comment 4•6 years ago
|
||
Extending a probe's lifetime requires a new data review.
Please file a data review request, see https://wiki.mozilla.org/Firefox/Data_Collection#Requesting_Data_Collection
Assignee | ||
Comment 5•6 years ago
|
||
Gah, was this changed? I remember getting told that I don't need this on previous data reviews. That wiki page also doesn't mention the case.
I'll request backout...
Comment 6•6 years ago
|
||
Backed out changeset 55be882db711 (Bug 1494589) at developer's request
Backout link: https://hg.mozilla.org/integration/autoland/rev/4b08331584c1c052189c41e4de02ef0edf5a2556
Comment 7•6 years ago
|
||
I have filed https://github.com/mozilla/data-review/issues/12 to track that it isn't clear from the wiki what does and doesn't require review.
For the record, all new and changed data collection requires review.
Updated•6 years ago
|
Attachment #9012521 -
Attachment description: Bug 1494589 - Extend PERMISSION_REQUEST_* probe lifetime. r=florian → Bug 1494589 - Extend PERMISSION_REQUEST_* probe lifetime. r=florian,data-r=chutten
Assignee | ||
Comment 8•6 years ago
|
||
Chris, are you going to sign this off or is there any issue?
Thanks!
Flags: needinfo?(chutten)
Comment 9•6 years ago
|
||
Where is the data collection review request form? Is it on another bug?
Flags: needinfo?(chutten) → needinfo?(jhofmann)
Assignee | ||
Comment 10•6 years ago
|
||
Flags: needinfo?(jhofmann)
Attachment #9015825 -
Flags: review?(chutten)
Comment 11•6 years ago
|
||
Comment on attachment 9015825 [details]
request.md
DATA COLLECTION REVIEW RESPONSE:
Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate?
Yes. Standard Telemetry mechanisms apply.
Is there a control mechanism that allows the user to turn the data collection on and off?
Yes. Standard Telemetry mechanisms apply.
If the request is for permanent data collection, is there someone who will monitor the data over time?
N/A, expires in 70.
Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
Category 2, Interaction
Is the data collection request for default-on or default-off?
Default on for prerelease only.
Does the instrumentation include the addition of any new identifiers (whether anonymous or otherwise; e.g., username, random IDs, etc. See the appendix for more details)?
No.
Is the data collection covered by the existing Firefox privacy notice?
Yes.
Does there need to be a check-in in the future to determine whether to renew the data?
Yes. :johannh is responsible for removing or renewing the probe before it expires in Firefox 70.
---
Result: datareview+
Attachment #9015825 -
Flags: review?(chutten) → review+
Assignee | ||
Comment 12•6 years ago
|
||
Thanks!
Comment 13•6 years ago
|
||
Pushed by jhofmann@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5162ce7f1c51
Extend PERMISSION_REQUEST_* probe lifetime. r=florian,data-r=chutten
Comment 14•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
You need to log in
before you can comment on or make changes to this bug.
Description
•