Page MenuHomePhabricator

Update OpenStackManager to use AuthManager
Closed, ResolvedPublic

Description

Needs a way to interact with LdapAuthentication when it is the primary provider.

Related Objects

StatusSubtypeAssignedTask
Resolved Deskana
ResolvedAnomie
OpenNone
ResolvedAnomie
OpenNone
ResolvedTgr
ResolvedAnomie
OpenFeatureNone
OpenNone
OpenNone
Resolved demon
ResolvedTgr
ResolvedNone
Resolved Mattflaschen-WMF
ResolvedTgr
ResolvedAnomie
ResolvedAnomie
ResolvedAnomie
Resolvedbd808

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.

The hard part is done with the merge of rEOSTca5602fbc403: Adjust to changes to LdapAuthentication in preparation for AuthManager. The one bit remaining is the use of the AbortNewAccount hook, which technically should be turned into a PreAuthenticationProvider but I'm somewhat inclined to just let LegacyHookPreAuthenticationProvider handle this one until OpenStackManager is killed.

Should we be lazy, or should we do the PreAuthenticationProvider?

Note that "do the PreAuthenticationProvider" probably also wants it converted to use extension.json so we can use a callback to check $wgDisableAuthManager at the right time.

I don't see much point in doing it if the extension is about to be abandoned anyway.

Anomie claimed this task.

Ok, let's declare victory here.

The one bit remaining is the use of the AbortNewAccount hook, which technically should be turned into a PreAuthenticationProvider but I'm somewhat inclined to just let LegacyHookPreAuthenticationProvider handle this one until OpenStackManager is killed.

That didn't work out because the extension used some LdapAuthentication proxy hooks (e.g. LDAPModifyUITemplate instead of UserLoginForm) and those got disabled when LdapAuthentication was updated.

Change 293130 had a related patch set uploaded (by Gergő Tisza):
Preserve shell account field for AuthManager

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

Change 293130 merged by jenkins-bot:
Preserve shell account field for AuthManager

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

Change 293431 had a related patch set uploaded (by Gergő Tisza):
Preserve shell account field for AuthManager

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

Change 293431 merged by jenkins-bot:
Preserve shell account field for AuthManager

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

Change 296515 had a related patch set uploaded (by Gergő Tisza):
Preserve shell account field for AuthManager

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

Change 296515 merged by jenkins-bot:
Preserve shell account field for AuthManager

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