Page MenuHomePhabricator

LoginNotify checks should be asynchronous
Closed, ResolvedPublic5 Story Points


In my admittedly unscientific testing, logging in an account present on every wiki takes ~5 seconds from enwiki and 8-9 seconds on testwiki. The difference is likely due to LoginNotify (specifically, it calling the brutally slow CentralAuthUser::queryAttached). The solution would be to move processing of failed logins to job queue.

Event Timeline

MaxSem created this task.Jun 12 2017, 8:30 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 12 2017, 8:30 PM

Let's make the cross-wiki checks happen in a deferred Job Queue job.

kaldari triaged this task as High priority.Jun 13 2017, 11:22 PM
kaldari set the point value for this task to 5.
MaxSem claimed this task.

Change 360787 had a related patch set uploaded (by Legoktm; owner: MaxSem):
[mediawiki/extensions/LoginNotify@master] Move expensive processing into job queue

Niharika closed this task as Resolved.