Page MenuHomePhabricator

CentralNotice: Slurp up existing wild cookies and copy them to LocalStorage or delete them
Open, Needs TriagePublic2 Estimated Story Points

Description

This task is about the cookie slurp code. For finalizing, communicating and deploying the actual list of wild slurp cookies, please see T135090.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 13 2016, 10:05 PM
AndyRussG renamed this task from CentralNotice: Slurp up existing wild cookies and copy them to LcoalStorage or delete them to CentralNotice: Slurp up existing wild cookies and copy them to LocalStorage or delete them.Apr 21 2016, 7:37 PM
AndyRussG moved this task from Backlog to Doing on the Fundraising Sprint Hermit Crab Husbandry board.
AndyRussG added a comment.EditedMay 6 2016, 7:25 PM

It seems the best approach here will be to have a configuration variable with a list of old cookies to delete. We'll run through the list and remove any we find on every pageview.

The vast majority of the cookies appear to be involved in counting banner impressions. Here is a tentative list, based on Mixin parameters in logs and cookies seen on production. (See: T132374#2229057, T131319#2197176 and T108849#2050638.)

Any cookies that are expected to be reused by future or existing campaigns should be left alone, since they'll be migrated automatically to LocalStorage (by code added for T132639) when such campaigns become active.

Change 287289 had a related patch set uploaded (by AndyRussG):
Slurp and delete defunct cookies on every pageview

https://gerrit.wikimedia.org/r/287289

Change 287289 merged by jenkins-bot:
Slurp and delete defunct cookies on every pageview

https://gerrit.wikimedia.org/r/287289

The code that allows this is now merged. I'm making a separate task for the finalizing the list of cookies to slurp, and the related config change, since some community consultation or announcement may well be needed.

AndyRussG updated the task description. (Show Details)May 12 2016, 1:38 AM

New task mentioned above (and now in this task's description): T135090.

Aklapper removed AndyRussG as the assignee of this task.Jun 19 2020, 4:16 PM

This task has been assigned to the same task owner for more than two years. Resetting task assignee due to inactivity, to decrease task cookie-licking and to get a slightly more realistic overview of plans. Please feel free to assign this task to yourself again if you still realistically work or plan to work on this task - it would be welcome!

For tips how to manage individual work in Phabricator (noisy notifications, lists of task, etc.), see https://phabricator.wikimedia.org/T228575#6237124 for available options.
(For the records, two emails were sent to assignee addresses before resetting assignees. See T228575 for more info and for potential feedback. Thanks!)