Anomie has uploaded a new change for review.
https://gerrit.wikimedia.org/r/291229
Change subject: Return a master instance from our implementation of
AuthPlugin::getUserInstance()
......................................................................
Return a master instance from our implementation of
AuthPlugin::getUserInstance()
We don't know if the caller is going to call write methods like
AuthPluginUser::resetAuthToken(), so to be safe we need to return a
master instance here.
Bug: T135954
Change-Id: Ic2a6c2a684f4787603ab13310e6b40c6471e3cd6
---
M includes/CentralAuthPlugin.php
1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CentralAuth
refs/changes/29/291229/1
diff --git a/includes/CentralAuthPlugin.php b/includes/CentralAuthPlugin.php
index 37dff76..f2a256b 100644
--- a/includes/CentralAuthPlugin.php
+++ b/includes/CentralAuthPlugin.php
@@ -364,7 +364,9 @@
* @return CentralAuthUser
*/
public function getUserInstance( User &$user ) {
- return CentralAuthUser::getInstance( $user );
+ // Needs to be a master instance because we don't know if the
caller is
+ // going to call write methods like
AuthPluginUser::resetAuthToken().
+ return CentralAuthUser::getMasterInstance( $user );
}
/**
--
To view, visit https://gerrit.wikimedia.org/r/291229
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic2a6c2a684f4787603ab13310e6b40c6471e3cd6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/CentralAuth
Gerrit-Branch: master
Gerrit-Owner: Anomie <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
