Page MenuHomePhabricator

Implement SUL3 central login for temp users
Closed, ResolvedPublic

Description

Previously we have said central login won't be used with SUL3, but actually temp users will still need it. So much like T375788: Implement SUL3 central autologin, its URL handling will need to be updated.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change #1078400 had a related patch set uploaded (by D3r1ck01; author: Derick Alangi):

[mediawiki/extensions/CentralAuth@master] Special: Add support for central auto login in SUL3 mode

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

Krinkle renamed this task from Implement SUL3 central login to Implement SUL3 central login for temp users.Oct 16 2024, 2:41 PM
Tgr added a subscriber: DAlangi_WMF.

Assigning to @DAlangi_WMF since the patch he is working on is covering this as well.

DAlangi_WMF changed the task status from Open to In Progress.Nov 19 2024, 9:50 AM

Change #1078400 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@master] Special: Add support for central autologin in SUL3 mode

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

@Tgr, with https://gerrit.wikimedia.org/r/c/1078400 merged, temp account central login for the shared domain works but is still sometimes affected by T377561: "Keep me logged in" flag unreliable on the central domain sometimes.

Technically speaking, this is done, maybe we should resolve this task and continue work on T377561: "Keep me logged in" flag unreliable on the central domain instead?

Change #1104635 had a related patch set uploaded (by D3r1ck01; author: Derick Alangi):

[mediawiki/extensions/CentralAuth@master] Trigger edge-login for temporary accounts in SUL3 mode

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

DAlangi_WMF changed the status of subtask T377561: "Keep me logged in" flag unreliable on the central domain from Open to In Progress.

Change #1104619 had a related patch set uploaded (by Gergő Tisza; author: Derick Alangi):

[mediawiki/extensions/CentralAuth@master] CentralAuthSessionProvider: Fix race condition in central login

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

Yeah I think this is done. I'm not sure the problem you are running into is the same as T377561 (which is about the browser preventing a request), but either way, it doesn't seem specific to SUL3. T377561 is definitely happening in production today.

Change #1104619 abandoned by D3r1ck01:

[mediawiki/extensions/CentralAuth@master] CentralAuthSessionProvider: Fix race condition in central login

Reason:

MediaWiki CLI's `mysql-replica` service (even with containers not running - stopped) seems to have caused this someone. Destroying the service entirely seems to have resolved this issue for me locally.

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

Change #1104635 abandoned by D3r1ck01:

[mediawiki/extensions/CentralAuth@master] Trigger edge-login for newly created temporary accounts

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