Page MenuHomePhabricator

Translate extension should not depend on deprecated ApiTokens module
Closed, ResolvedPublic1 Estimated Story Points

Description

It can still use this module for pre MediaWiki 1.24 installs (if desired) but when installed in current MediaWiki it should use csrf tokens.

Relevant:

Event Timeline

Krinkle assigned this task to Nikerabbit.
Krinkle raised the priority of this task from to High.
Krinkle updated the task description. (Show Details)
Krinkle added subscribers: Anomie, Aklapper, Legoktm, Krinkle.
Krinkle renamed this task from Translate extension should not depend deprecated ApiTokens module to Translate extension should not depend on deprecated ApiTokens module.Nov 4 2015, 8:17 PM
Krinkle set Security to None.

With 42e9a653090288d9b57225130c206a0559325131 landed, the old interface now supports csrf as well so that Translate can migrate without having a circular dependency on MediaWiki core migrating to the new interface first.

With 42e9a653090288d9b57225130c206a0559325131 landed, the old interface now supports csrf as well so that Translate can migrate without having a circular dependency on MediaWiki core migrating to the new interface first.

I don't see how that helps me. I used edit as alias for csrf for now while getting rid of the old custom token types.

Change 251697 had a related patch set uploaded (by Nikerabbit):
Clean up token handling

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

With 42e9a653090288d9b57225130c206a0559325131 landed, the old interface now supports csrf as well so that Translate can migrate without having a circular dependency on MediaWiki core migrating to the new interface first.

I don't see how that helps me. I used edit as alias for csrf for now while getting rid of the old custom token types.

That's fine too. I thought you would want to use "csrf" directly in getToken(), which didn't work in the past because it only existed in the new API.

Change 251697 merged by jenkins-bot:
Clean up token handling

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