Page MenuHomePhabricator

Update DisableAccount to use AuthManager
Closed, ResolvedPublic

Description

See T110433 and T110291.

Items needing updating:

  • Uses $user->setPassword( null ) - Since this is supposed to be very permanent, use SessionManager::preventSessionsForUser() and AuthManager::revokeAccessForUser(). Or maybe just call User::newSystemUser( $name, [ 'steal' => true ] ).

Related Objects

Event Timeline

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

Change 289706 had a related patch set uploaded (by Anomie):
(Ab)Use User::newSystemUser to disable accounts

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

Change 289706 merged by jenkins-bot:
(Ab)Use User::newSystemUser to disable accounts

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

Anomie claimed this task.

Change 296510 had a related patch set uploaded (by Gergő Tisza):
(Ab)Use User::newSystemUser to disable accounts

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

Change 296510 merged by jenkins-bot:
(Ab)Use User::newSystemUser to disable accounts

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