Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Rename 'authmanager' log channel to 'authevents'

Browse source 
Also, move some events which did not belong there to the
'authentication' channel.

Change-Id: I4d7564343318899fc498ed2bd2d5260041ee22cc
This commit is contained in:
Gergő Tisza 2016-08-10 01:32:28 +00:00
parent 0f91f711b5
commit df342ebda5
7 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
includes/Setup.php
View file

@ -840,7 +840,7 @@ if ( !defined( 'MW_NO_SESSION' ) && !$wgCommandLineMode ) {
true
);
Profiler::instance()->scopedProfileOut( $ps_autocreate );
\MediaWiki\Logger\LoggerFactory::getInstance( 'authmanager' )->info( 'Autocreation attempt', [
\MediaWiki\Logger\LoggerFactory::getInstance( 'authevents' )->info( 'Autocreation attempt', [
'event' => 'autocreate',
'status' => $res,
] );

2
includes/api/ApiAuthManagerHelper.php
View file

@ -238,7 +238,7 @@ class ApiAuthManagerHelper {
}
$module = $this->module->getModuleName();
LoggerFactory::getInstance( 'authmanager' )->info( "$module API attempt", [
LoggerFactory::getInstance( 'authevents' )->info( "$module API attempt", [
'event' => $event,
'status' => $status,
'module' => $module,

4
includes/api/ApiLogin.php
View file

@ -115,7 +115,7 @@ class ApiLogin extends ApiBase {
} else {
$authRes = 'Failed';
$message = $status->getMessage();
LoggerFactory::getInstance( 'authmanager' )->info(
LoggerFactory::getInstance( 'authentication' )->info(
'BotPassword login failed: ' . $status->getWikiText( false, false, 'en' )
);
}
@ -226,7 +226,7 @@ class ApiLogin extends ApiBase {
if ( $loginType === 'LoginForm' && isset( LoginForm::$statusCodes[$authRes] ) ) {
$authRes = LoginForm::$statusCodes[$authRes];
}
LoggerFactory::getInstance( 'authmanager' )->info( 'Login attempt', [
LoggerFactory::getInstance( 'authevents' )->info( 'Login attempt', [
'event' => 'login',
'successful' => $authRes === 'Success',
'loginType' => $loginType,

2
includes/specialpage/AuthManagerSpecialPage.php
View file

@ -455,7 +455,7 @@ abstract class AuthManagerSpecialPage extends SpecialPage {
// passed to AuthManager. Normally we would display the form with an error message,
// but for the data we received via the redirect flow that would not be helpful at all.
// Let's just submit the data to AuthManager directly instead.
LoggerFactory::getInstance( 'authmanager' )
LoggerFactory::getInstance( 'authentication' )
->warning( 'Validation error on return', [ 'data' => $form->mFieldData,
'status' => $status->getWikiText() ] );
$status = $this->handleFormSubmit( $form->mFieldData );

2
includes/specials/SpecialCreateAccount.php
View file

@ -159,7 +159,7 @@ class SpecialCreateAccount extends LoginSignupSpecialPage {
}
protected function logAuthResult( $success, $status = null ) {
LoggerFactory::getInstance( 'authmanager' )->info( 'Account creation attempt', [
LoggerFactory::getInstance( 'authevents' )->info( 'Account creation attempt', [
'event' => 'accountcreation',
'successful' => $success,
'status' => $status,

2
includes/specials/SpecialUserLogin.php
View file

@ -153,7 +153,7 @@ class SpecialUserLogin extends LoginSignupSpecialPage {
}
protected function logAuthResult( $success, $status = null ) {
LoggerFactory::getInstance( 'authmanager' )->info( 'Login attempt', [
LoggerFactory::getInstance( 'authevents' )->info( 'Login attempt', [
'event' => 'login',
'successful' => $success,
'status' => $status,

2
includes/user/User.php
View file

@ -3768,7 +3768,7 @@ class User implements IDBAccessObject {
ScopedCallback::consume( $delay );
$error = false;
}
\MediaWiki\Logger\LoggerFactory::getInstance( 'authmanager' )->info( 'Logout', [
\MediaWiki\Logger\LoggerFactory::getInstance( 'authevents' )->info( 'Logout', [
'event' => 'logout',
'successful' => $error === false,
'status' => $error ?: 'success',