Теперь на этапе включения TOTP код может истечь в течение 2 периодов

This commit is contained in:
ErickSkrauch 2017-11-20 20:48:43 +03:00
parent dab0ab4cba
commit 47b6761f78
3 changed files with 13 additions and 2 deletions

View File

@ -18,7 +18,7 @@ class EnableTwoFactorAuthForm extends AccountActionForm {
return [ return [
['account', 'validateOtpDisabled'], ['account', 'validateOtpDisabled'],
['totp', 'required', 'message' => E::TOTP_REQUIRED], ['totp', 'required', 'message' => E::TOTP_REQUIRED],
['totp', TotpValidator::class, 'account' => $this->getAccount()], ['totp', TotpValidator::class, 'account' => $this->getAccount(), 'window' => 2],
['password', PasswordRequiredValidator::class, 'account' => $this->getAccount()], ['password', PasswordRequiredValidator::class, 'account' => $this->getAccount()],
]; ];
} }

View File

@ -20,7 +20,7 @@ class TotpValidator extends Validator {
* @var int|null Задаёт окно, в промежуток которого будет проверяться код. * @var int|null Задаёт окно, в промежуток которого будет проверяться код.
* Позволяет избежать ситуации, когда пользователь ввёл код в последнюю секунду * Позволяет избежать ситуации, когда пользователь ввёл код в последнюю секунду
* его существования и пока шёл запрос, тот протух. * его существования и пока шёл запрос, тот протух.
* Значение задаётся в +- кодах, а не секундах. * Значение задаётся в +- периодах, а не секундах.
*/ */
public $window; public $window;

View File

@ -58,4 +58,15 @@ class EnableTwoFactorAuthCest {
]); ]);
} }
public function testSuccessEnableWithNotSoExpiredCode(FunctionalTester $I) {
$accountId = $I->amAuthenticated('AccountWithOtpSecret');
$totp = TOTP::create('AAAA');
$this->route->enableTwoFactorAuth($accountId, $totp->at(time() - 35), 'password_0');
$I->canSeeResponseCodeIs(200);
$I->canSeeResponseIsJson();
$I->canSeeResponseContainsJson([
'success' => true,
]);
}
} }