Remove unused fields in the Pocket impression ping
Categories
(Firefox :: New Tab Page, enhancement, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox86 | --- | verified |
People
(Reporter: nanj, Assigned: nanj)
References
Details
Attachments
(1 file)
As reported in bug 1682360 and bug 1682366, various fields (client_id
, session_id
, action
) in the Pocket impression ping & SPOCS fill are not tracked by the table schema. Most of them are not applicable and set to "n/a", we should remove those fields from the payload so the sent pings are matching against the schema.
Updated•3 years ago
|
Assignee | ||
Comment 1•3 years ago
|
||
Kirill or Scot, are we still using the spocs_fill ping? If not, perhaps we'd better deprecate this ping now.
Assignee | ||
Comment 3•3 years ago
|
||
Sounds good! Will file a separate bug to track that.
Assignee | ||
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Comment 5•3 years ago
|
||
Pushed by najiang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bcfbf7c9d108 Remove unused fields in the impression stats ping r=thecount
Comment 7•3 years ago
|
||
bugherder |
Comment 8•3 years ago
|
||
I have verified this enhancement and I can confirm that the client_id
, session_id
, and action
fields are no longer displayed inside the "Pocket Impression" ping from the "Browser Console".
Verified using the latest Firefox Nightly (86.0a1 Build ID - 20210118094603) installed on Windows 10 x64, Mac 10.15.7, and Linux Mint 20 x64.
Description
•