BryanDavis has uploaded a new change for review. https://gerrit.wikimedia.org/r/318036
Change subject: wikitech: Set wgMWOAuthCentralWiki = false ...................................................................... wikitech: Set wgMWOAuthCentralWiki = false MWOauthHooks::onSetupAfterCache will set $wgMWOAuthSharedUserIDs = true if $wgMWOAuthCentralWiki is anything other than false. This causes an extra indirection through CentralIdLookup when creating the user in MWOAuthUtils::getCentralUserNameFromId which is unneeded. As an unintended side effect it will also break things by attempting to verify the OAuth tokens and nonce twice. Bug: T149150 Change-Id: I38e72794c4830a97fecde58d097d694e8c4ef1aa --- M wmf-config/CommonSettings.php 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/operations/mediawiki-config refs/changes/36/318036/1 diff --git a/wmf-config/CommonSettings.php b/wmf-config/CommonSettings.php index a6d245d..2961682 100644 --- a/wmf-config/CommonSettings.php +++ b/wmf-config/CommonSettings.php @@ -3105,7 +3105,7 @@ wfLoadExtension( 'OAuth' ); if ( in_array( $wgDBname, [ 'labswiki', 'labtestwiki' ] ) ) { // Wikitech and its testing variant use local OAuth tables - $wgMWOAuthCentralWiki = $wgDBname; + $wgMWOAuthCentralWiki = false; } else { $wgMWOAuthCentralWiki = 'metawiki'; $wgMWOAuthSharedUserSource = 'CentralAuth'; -- To view, visit https://gerrit.wikimedia.org/r/318036 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I38e72794c4830a97fecde58d097d694e8c4ef1aa Gerrit-PatchSet: 1 Gerrit-Project: operations/mediawiki-config Gerrit-Branch: master Gerrit-Owner: BryanDavis <bda...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits