Page MenuHomePhabricator

VisualEditor: Remove obsolete retry token logic in mw.Target
Closed, ResolvedPublic

Description


Version: unspecified
Severity: normal

Details

Reference
bz51253

Event Timeline

bzimport raised the priority of this task from to Normal.Nov 22 2014, 1:49 AM
bzimport set Reference to bz51253.
Krinkle created this task.Jul 12 2013, 6:49 PM

onLoad no longer requires or even considers a token, so this is now obsolete.

It also ignored the fact that the new token may be logged out or for a different user, though this was previous OK-ish because it was only to load the content which is a read action and not a write action (hence why we fixed that API recently to no longer require a token)

Change 76860 had a related patch set uploaded by Krinkle:
Removed unreachable badtoken recovery code

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

Change 76860 merged by jenkins-bot:
mw.Target: Removed unreachable badtoken recovery code

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