Centralize context_id reading mechanism into a module, and add the ability to rotate the context_id after some number of days
Categories
(Firefox :: New Tab Page, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox140 | --- | fixed |
People
(Reporter: mconley, Assigned: mconley)
References
(Blocks 2 open bugs)
Details
Attachments
(4 files, 3 obsolete files)
We have a number of places where we read the context ID preference:
https://searchfox.org/mozilla-central/search?q=.contextId%22&path=&case=false®exp=false
It's quite repetitive, and we can probably centralize this a bit. Similarly, I've been talking to efixler from the ads team, and we want to try rotating the context_id every 30 days. Rotating the context_id was, apparently, always the plan, but until now nobody's filed a bug to do it. So I'm hoping this can be a two-fer.
I'm also planning on making the rotation window instrumentable from Nimbus so that we can shrink it further from 30 days, if we decide we'd like to do that down the road.
Assignee | ||
Comment 1•5 months ago
|
||
Updated•5 months ago
|
Assignee | ||
Comment 2•5 months ago
|
||
Assignee | ||
Comment 3•5 months ago
|
||
Comment 4•4 months ago
|
||
The following patch is waiting for review from an inactive reviewer:
ID | Title | Author | Reviewer Status |
---|---|---|---|
D241480 | Bug 1952316 - Have Search / URL bar code use the ContextID module for accessing the current context_id. r?adw!,Dexter! | mconley | adw: Back May 12, 2025 |
:mconley, could you please find another reviewer?
For more information, please visit BugBot documentation.
Assignee | ||
Comment 5•4 months ago
|
||
I'm re-implementing this work as a shared Rust component in application-services, and will use this bug to vendor that component in and wrap the Uniffi-generated JS with something a little more ergonomic.
Assignee | ||
Comment 6•3 months ago
|
||
Assignee | ||
Comment 7•3 months ago
|
||
Assignee | ||
Comment 8•3 months ago
|
||
Assignee | ||
Comment 9•3 months ago
|
||
Updated•3 months ago
|
Updated•3 months ago
|
Updated•3 months ago
|
Assignee | ||
Comment 10•3 months ago
|
||
Okay, I think I'm finally ready for review here. The end goal with this stack of patches is, by default, that the actual behaviour of what gets sent for Glean ingestion / MARS does not change with the default prefs here - but then we rotation is enabled (which requires both enabling the Rust backend, and having the rotationDays value be greater than 0), that sending the context ID to MARS and Glean for deletion is delegated entirely to the ContextId.sys.mjs module.
I realize there's additional work required here for the ETL job for the context-id-deletion-request ping.
Updated•3 months ago
|
Comment 11•3 months ago
|
||
Comment 12•3 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/bf2511b7937a
https://hg.mozilla.org/mozilla-central/rev/2861c19f2109
https://hg.mozilla.org/mozilla-central/rev/d9de52a3711b
https://hg.mozilla.org/mozilla-central/rev/955c37619ae6
Updated•3 months ago
|
Description
•