Aaron Schulz has uploaded a new change for review.
https://gerrit.wikimedia.org/r/237976
Change subject: Made attemptAddUser() explicity make sure the central DB is up
......................................................................
Made attemptAddUser() explicity make sure the central DB is up
* Once slaves are used more, this check will make a difference
in terms of handling central DB master outages
Change-Id: I05c93ffdd417a1e7ec10040252c9a7742fa25e18
---
M includes/CentralAuthHooks.php
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CentralAuth
refs/changes/76/237976/1
diff --git a/includes/CentralAuthHooks.php b/includes/CentralAuthHooks.php
index 18d256b..ad62f64 100644
--- a/includes/CentralAuthHooks.php
+++ b/includes/CentralAuthHooks.php
@@ -1193,6 +1193,9 @@
$from = isset( $_SERVER['REQUEST_URI'] ) ?
$_SERVER['REQUEST_URI'] : 'CLI';
wfDebugLog( 'CentralAuth-Bug39996', __METHOD__ . ": creating
new user ($userName) - from: $from\n" );
try {
+ // Make sure the central DB master is availabe
+ CentralAuthUser::getCentralDB();
+ // Insert the user into the local DB master
$status = $user->addToDatabase();
} catch ( Exception $e ) {
wfDebugLog( 'CentralAuth-Bug39996', __METHOD__ . "
User::addToDatabase for \"$userName\" threw an exception:"
--
To view, visit https://gerrit.wikimedia.org/r/237976
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I05c93ffdd417a1e7ec10040252c9a7742fa25e18
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/CentralAuth
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
