mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/ConfirmEdit
synced 2024-12-21 04:13:03 +00:00
2dfc290c57
isBadLoginPerUserTriggered() can never return null. This comment was
added in 2016 in 31c59374a4
and it was
already incorrect then. I don't know where this idea came from.
Change-Id: Ib919999fe83562cb4fa80246ae7c6b4707da775c
193 lines
6.5 KiB
PHP
193 lines
6.5 KiB
PHP
<?php
|
|
|
|
namespace MediaWiki\Extension\ConfirmEdit\Auth;
|
|
|
|
use MediaWiki\Auth\AbstractPreAuthenticationProvider;
|
|
use MediaWiki\Auth\AuthenticationRequest;
|
|
use MediaWiki\Auth\AuthenticationResponse;
|
|
use MediaWiki\Auth\AuthManager;
|
|
use MediaWiki\Extension\ConfirmEdit\Hooks;
|
|
use MediaWiki\Extension\ConfirmEdit\SimpleCaptcha\SimpleCaptcha;
|
|
use MediaWiki\Logger\LoggerFactory;
|
|
use Status;
|
|
use User;
|
|
|
|
class CaptchaPreAuthenticationProvider extends AbstractPreAuthenticationProvider {
|
|
/**
|
|
* @inheritDoc
|
|
*/
|
|
public function getAuthenticationRequests( $action, array $options ) {
|
|
$captcha = Hooks::getInstance();
|
|
$user = User::newFromName( $options['username'] );
|
|
|
|
$needed = false;
|
|
switch ( $action ) {
|
|
case AuthManager::ACTION_CREATE:
|
|
$u = $user ?: new User();
|
|
$needed = $captcha->needCreateAccountCaptcha( $u );
|
|
if ( $needed ) {
|
|
$captcha->setAction( 'accountcreate' );
|
|
// This is debug level simply because generally
|
|
// captchas are either always or never triggered on
|
|
// view of create account, so it gets pretty noisy
|
|
LoggerFactory::getInstance( 'captcha' )
|
|
->debug( 'Captcha shown on account creation for {user}', [
|
|
'event' => 'captcha.display',
|
|
'eventType' => 'accountcreation',
|
|
'user' => $u->getName()
|
|
] );
|
|
}
|
|
break;
|
|
case AuthManager::ACTION_LOGIN:
|
|
// Captcha is shown on login when there were too many failed attempts from the
|
|
// current IP or user. The latter is a bit awkward because we don't know the
|
|
// username yet. The username from the last successful login is stored in a cookie,
|
|
// but we still must make sure to not lock out other usernames so we use a session
|
|
// flag. This will result in confusing error messages if the browser cannot persist
|
|
// the session, but then login would be impossible anyway so no big deal.
|
|
|
|
// If the username ends to be one that does not trigger the captcha, that will
|
|
// result in weird behavior (if the user leaves the captcha field open, they get
|
|
// a required field error, if they fill it with an invalid answer, it will pass)
|
|
// - again, not a huge deal.
|
|
$session = $this->manager->getRequest()->getSession();
|
|
$sessionFlag = $session->get( 'ConfirmEdit:loginCaptchaPerUserTriggered' );
|
|
$suggestedUsername = $session->suggestLoginUsername();
|
|
if (
|
|
$captcha->isBadLoginTriggered()
|
|
|| $sessionFlag
|
|
|| $suggestedUsername && $captcha->isBadLoginPerUserTriggered( $suggestedUsername )
|
|
) {
|
|
$needed = true;
|
|
$captcha->setAction( 'badlogin' );
|
|
LoggerFactory::getInstance( 'captcha' )
|
|
->info( 'Captcha shown on login by {clientip} for {suggestedUser}', [
|
|
'event' => 'captcha.display',
|
|
'eventType' => 'accountcreation',
|
|
'suggestedUser' => $suggestedUsername,
|
|
'clientip' => $this->manager->getRequest()->getIP()
|
|
] );
|
|
break;
|
|
}
|
|
break;
|
|
}
|
|
|
|
if ( $needed ) {
|
|
return [ $captcha->createAuthenticationRequest() ];
|
|
} else {
|
|
return [];
|
|
}
|
|
}
|
|
|
|
/**
|
|
* @inheritDoc
|
|
*/
|
|
public function testForAuthentication( array $reqs ) {
|
|
$captcha = Hooks::getInstance();
|
|
$username = AuthenticationRequest::getUsernameFromRequests( $reqs );
|
|
$success = true;
|
|
$isBadLoginPerUserTriggered = $username ?
|
|
$captcha->isBadLoginPerUserTriggered( $username ) : false;
|
|
|
|
if ( $captcha->isBadLoginTriggered() || $isBadLoginPerUserTriggered ) {
|
|
$captcha->setAction( 'badlogin' );
|
|
$captcha->setTrigger( "post-badlogin login '$username'" );
|
|
$success = $this->verifyCaptcha( $captcha, $reqs, new User() );
|
|
$ip = $this->manager->getRequest()->getIP();
|
|
LoggerFactory::getInstance( 'captcha' )->info( 'Captcha submitted on login for {user}', [
|
|
'event' => 'captcha.submit',
|
|
'eventType' => 'login',
|
|
'successful' => $success,
|
|
'user' => $username,
|
|
'clientip' => $ip
|
|
] );
|
|
}
|
|
|
|
if ( $isBadLoginPerUserTriggered ) {
|
|
$session = $this->manager->getRequest()->getSession();
|
|
$session->set( 'ConfirmEdit:loginCaptchaPerUserTriggered', true );
|
|
}
|
|
|
|
// Make brute force attacks harder by not telling whether the password or the
|
|
// captcha failed.
|
|
return $success ? Status::newGood() : $this->makeError( 'wrongpassword', $captcha );
|
|
}
|
|
|
|
/**
|
|
* @inheritDoc
|
|
*/
|
|
public function testForAccountCreation( $user, $creator, array $reqs ) {
|
|
$captcha = Hooks::getInstance();
|
|
|
|
if ( $captcha->needCreateAccountCaptcha( $creator ) ) {
|
|
$username = $user->getName();
|
|
$captcha->setAction( 'accountcreate' );
|
|
$captcha->setTrigger( "new account '$username'" );
|
|
$success = $this->verifyCaptcha( $captcha, $reqs, $user );
|
|
$ip = $this->manager->getRequest()->getIP();
|
|
LoggerFactory::getInstance( 'captcha' )->info(
|
|
'Captcha submitted on account creation for {user}', [
|
|
'event' => 'captcha.submit',
|
|
'eventType' => 'accountcreation',
|
|
'successful' => $success,
|
|
'user' => $username,
|
|
'clientip' => $ip
|
|
]
|
|
);
|
|
if ( !$success ) {
|
|
return $this->makeError( 'captcha-createaccount-fail', $captcha );
|
|
}
|
|
}
|
|
return Status::newGood();
|
|
}
|
|
|
|
/**
|
|
* @inheritDoc
|
|
*/
|
|
public function postAuthentication( $user, AuthenticationResponse $response ) {
|
|
$captcha = Hooks::getInstance();
|
|
switch ( $response->status ) {
|
|
case AuthenticationResponse::PASS:
|
|
case AuthenticationResponse::RESTART:
|
|
$session = $this->manager->getRequest()->getSession();
|
|
$session->remove( 'ConfirmEdit:loginCaptchaPerUserTriggered' );
|
|
$captcha->resetBadLoginCounter( $user ? $user->getName() : null );
|
|
break;
|
|
case AuthenticationResponse::FAIL:
|
|
$captcha->increaseBadLoginCounter( $user ? $user->getName() : null );
|
|
break;
|
|
}
|
|
}
|
|
|
|
/**
|
|
* Verify submitted captcha.
|
|
* Assumes that the user has to pass the capctha (permission checks are caller's responsibility).
|
|
* @param SimpleCaptcha $captcha
|
|
* @param AuthenticationRequest[] $reqs
|
|
* @param User $user
|
|
* @return bool
|
|
*/
|
|
protected function verifyCaptcha( SimpleCaptcha $captcha, array $reqs, User $user ) {
|
|
/** @var CaptchaAuthenticationRequest $req */
|
|
$req = AuthenticationRequest::getRequestByClass( $reqs,
|
|
CaptchaAuthenticationRequest::class, true );
|
|
if ( !$req ) {
|
|
return false;
|
|
}
|
|
return $captcha->passCaptchaLimited( $req->captchaId, $req->captchaWord, $user );
|
|
}
|
|
|
|
/**
|
|
* @param string $message Message key
|
|
* @param SimpleCaptcha $captcha
|
|
* @return Status
|
|
*/
|
|
protected function makeError( $message, SimpleCaptcha $captcha ) {
|
|
$error = $captcha->getError();
|
|
if ( $error ) {
|
|
return Status::newFatal( wfMessage( 'captcha-error', $error ) );
|
|
}
|
|
return Status::newFatal( $message );
|
|
}
|
|
}
|