$this->setSessionDataForUser( $user );
$this->callMethodOnProviders( 7, 'postAuthentication', [ $user, $ret ] );
$session->remove( 'AuthManager::authnState' );
- \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $ret, $user, $user->getName() ] );
+ \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $ret, $user, $user->getName(), [] ] );
return $ret;
}
$this->callMethodOnProviders( 7, 'postAuthentication',
[ User::newFromName( $guessUserName ) ?: null, $ret ]
);
- \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $ret, null, $guessUserName ] );
+ \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $ret, null, $guessUserName, [] ] );
return $ret;
}
}
[ User::newFromName( $guessUserName ) ?: null, $res ]
);
$session->remove( 'AuthManager::authnState' );
- \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $res, null, $guessUserName ] );
+ \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $res, null, $guessUserName, [] ] );
return $res;
case AuthenticationResponse::ABSTAIN;
// Continue loop
[ User::newFromName( $guessUserName ) ?: null, $res ]
);
$session->remove( 'AuthManager::authnState' );
- \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $res, null, $guessUserName ] );
+ \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $res, null, $guessUserName, [] ] );
return $res;
case AuthenticationResponse::REDIRECT;
case AuthenticationResponse::UI;
);
$this->callMethodOnProviders( 7, 'postAuthentication', [ $user, $ret ] );
$session->remove( 'AuthManager::authnState' );
- \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $ret, $user, $user->getName() ] );
+ \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $ret, $user, $user->getName(), [] ] );
return $ret;
}
}
$this->logger->debug( "Login failed in secondary authentication by $id" );
$this->callMethodOnProviders( 7, 'postAuthentication', [ $user, $res ] );
$session->remove( 'AuthManager::authnState' );
- \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $res, $user, $user->getName() ] );
+ \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $res, $user, $user->getName(), [] ] );
return $res;
case AuthenticationResponse::REDIRECT;
case AuthenticationResponse::UI;
// Step 4: Authentication complete! Set the user in the session and
// clean up.
- $this->logger->info( 'Login for {user} succeeded', [
+ $this->logger->info( 'Login for {user} succeeded from {clientip}', [
'user' => $user->getName(),
+ 'clientip' => $this->request->getIP(),
] );
/** @var RememberMeAuthenticationRequest $req */
$req = AuthenticationRequest::getRequestByClass(
$this->callMethodOnProviders( 7, 'postAuthentication', [ $user, $ret ] );
$session->remove( 'AuthManager::authnState' );
$this->removeAuthenticationSessionData( null );
- \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $ret, $user, $user->getName() ] );
+ \Hooks::run( 'AuthManagerLoginAuthenticateAudit', [ $ret, $user, $user->getName(), [] ] );
return $ret;
} catch ( \Exception $ex ) {
$session->remove( 'AuthManager::authnState' );
$user->load( User::READ_LOCKING );
if ( $state['userid'] === 0 ) {
- if ( $user->getId() != 0 ) {
+ if ( $user->getId() !== 0 ) {
$this->logger->debug( __METHOD__ . ': User exists locally', [
'user' => $user->getName(),
'creator' => $creator->getName(),
return $ret;
}
} else {
- if ( $user->getId() == 0 ) {
+ if ( $user->getId() === 0 ) {
$this->logger->debug( __METHOD__ . ': User does not exist locally when it should', [
'user' => $user->getName(),
'creator' => $creator->getName(),
"User \"{$state['username']}\" should exist now, but doesn't!"
);
}
- if ( $user->getId() != $state['userid'] ) {
+ if ( $user->getId() !== $state['userid'] ) {
$this->logger->debug( __METHOD__ . ': User ID/name mismatch', [
'user' => $user->getName(),
'creator' => $creator->getName(),
] );
throw new \UnexpectedValueException(
"User \"{$state['username']}\" exists, but " .
- "ID {$user->getId()} != {$state['userid']}!"
+ "ID {$user->getId()} !== {$state['userid']}!"
);
}
}
// @codeCoverageIgnoreStart
if (
!$localId &&
- MediaWikiServices::getInstance()->getDBLoadBalancer()->getReaderIndex() != 0
+ MediaWikiServices::getInstance()->getDBLoadBalancer()->getReaderIndex() !== 0
) {
$localId = User::idFromName( $username, User::READ_LATEST );
$flags = User::READ_LATEST;
$session->remove( 'AuthManager::accountLinkState' );
return AuthenticationResponse::newFail( wfMessage( 'noname' ) );
}
- if ( $user->getId() != $state['userid'] ) {
+ if ( $user->getId() !== $state['userid'] ) {
throw new \UnexpectedValueException(
"User \"{$state['username']}\" is valid, but " .
- "ID {$user->getId()} != {$state['userid']}!"
+ "ID {$user->getId()} !== {$state['userid']}!"
);
}