BryanDavis has uploaded a new change for review.

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

Change subject: Add statsd metrics for SUL migration renamed accounts
......................................................................

Add statsd metrics for SUL migration renamed accounts

Track the number of times that force renamed SUL accounts are checked
and coerced with a StatsD if configured. Metrics monitored are:
* MediaWiki.centralauth.migration.check: checked to see if password was
  really for USER~wiki instead of USER
* MediaWiki.centralauth.migration.coerce: switched authenticated user
  from USER to USER~wiki

Change-Id: If7d87fe63b714e88b743d3c54f9282f9b521d288
---
M includes/CentralAuthPlugin.php
1 file changed, 6 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CentralAuth 
refs/changes/33/247933/1

diff --git a/includes/CentralAuthPlugin.php b/includes/CentralAuthPlugin.php
index 3542090..6d10864 100644
--- a/includes/CentralAuthPlugin.php
+++ b/includes/CentralAuthPlugin.php
@@ -57,6 +57,9 @@
                                wfDebugLog( 'CentralAuth',
                                        "CentralAuthMigration: Checking for 
migration of '{$username}' to '{$renamedUsername}'"
                                );
+                               
RequestContext::getMain()->getStats()->increment(
+                                       'centralauth.migration.check'
+                               );
 
                                $renamed = new CentralAuthUser( 
$renamedUsername );
                                $passwordMatch = self::checkPassword( $renamed, 
$password );
@@ -190,6 +193,9 @@
                        wfDebugLog( 'CentralAuth',
                                "CentralAuthMigration: Coercing user to 
'{$this->sulMigrationName}'"
                        );
+                       RequestContext::getMain()->getStats()->increment(
+                               'centralauth.migration.coerce'
+                       );
 
                        // Create a new user object using the post-migration 
name
                        $user = User::newFromName( $this->sulMigrationName );

-- 
To view, visit https://gerrit.wikimedia.org/r/247933
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If7d87fe63b714e88b743d3c54f9282f9b521d288
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/CentralAuth
Gerrit-Branch: master
Gerrit-Owner: BryanDavis <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to