Page MenuHomePhabricator

Move CentralNotice stuff out of cookies
Open, HighPublic

Description

@ori mentioned that we should try to cut down on our cookie habit, and put stuff in localstorage instead. Apparently the ResourceLoader issue was fixed, that was filling the storage.

Cookies are expensive because they're sent back and forth on every pageview.

Event Timeline

awight raised the priority of this task from to Needs Triage.
awight updated the task description. (Show Details)
awight added subscribers: awight, ori.

There is some progress! Unfortunately there are two big areas still to work on for this: bucket cookies and hide cookies. While we do have a viable LocalStorage-based replacement mechanism in place, we also need to check what sort of fallback mechanism, if any, would be needed, at least for Fundraising.

ori triaged this task as High priority.Feb 22 2016, 9:12 AM

There is some progress! Unfortunately there are two big areas still to work on for this: bucket cookies and hide cookies. While we do have a viable LocalStorage-based replacement mechanism in place, we also need to check what sort of fallback mechanism, if any, would be needed, at least for Fundraising.

@AndyRussG, what is the status of this work? I just checked and on enwiki I currently have eight(!) cookies from CentralNotice:

  1. CN
  2. HarassmentSurvey_A
  3. HarassmentSurvey_A-wait
  4. bannercount_fundraiser_Dec2015
  5. bannercount_fundraiser_Dec2015-wait
  6. centralnotice-frbanner-seen-fullscreen
  7. centralnotice_bannercount_fr15
  8. centralnotice_bannercount_fr15-wait

@ori thx for the ping! It's not currently programmed for the near future, but I think we should be able to bring it in. I'll talk to folks and get back to you. :)

@ori thx for the ping! It's not currently programmed for the near future, but I think we should be able to bring it in. I'll talk to folks and get back to you. :)

Thanks! Let me know if I can help.

DStrine set the point value for this task to 4.

I'd love to what proportion of our visitors have LocalStorage-capable browsers. Falling back to cookies for the ones that don't support LS seems problematic because these are probably older systems to begin with. Are we considering a non-cookie fallback?

Are we logging local storage failures now? Browser support is looking pretty good on caniuse, but I dunno how many people disable it via some method. http://caniuse.com/#feat=namevalue-storage. Total Opera mobile looks like less than 1% of our views on https://stats.wikimedia.org/wikimedia/squids/SquidReportClients.htm

AndyRussG removed the point value 4 for this task.Apr 13 2016, 10:05 PM

Removed story points because this is now a tracking task (See blocked-by for subtasks.)