Page MenuHomePhabricator

fix schema MobileWebSectionUsage
Closed, ResolvedPublic

Description

Event Timeline

kevinator assigned this task to JKatzWMF.
kevinator raised the priority of this task from to Needs Triage.
kevinator updated the task description. (Show Details)
kevinator added a project: Mobile.
JKatzWMF added subscribers: Jdlrobson, Jhernandez, JKatzWMF.

@Jdlrobson welcome back! I think this is something you should probably manage. @Jhernandez, I am going to throw this in sprint 62 tentatively, in case JR confirms it is an issue and we need to pull it ASAP

Change 255796 had a related patch set uploaded (by BryanDavis):
getSessionId's value should only last for session duration

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

Change 255796 merged by jenkins-bot:
getSessionId's value should only last for session duration

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

bd808 triaged this task as Medium priority.Dec 2 2015, 3:45 AM
bd808 moved this task from To Do to Done on the User-bd808 board.

I've left a comment to the patch. I've also tested it on the beta cluster and as expected the localStorage sessionId is not being set and the cookie session ID expires once the user quits the browser.

Change 257296 had a related patch set uploaded (by Phuedx):
Remove previously stored session IDs

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

Change 257296 merged by jenkins-bot:
Remove previously stored session IDs

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