Leisuretimedock/blessing-skin-server
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix validating captcha

Browse Source
This commit is contained in:
Pig Fang 2018-07-21 08:17:16 +08:00
parent 201fe21e6d
commit 8bd60624a0
2 changed files with 10 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
app/Http/Controllers/AuthController.php
View File

@ -40,8 +40,7 @@ class AuthController extends Controller
$user = $users->get($identification, $authType); $user = $users->get($identification, $authType);
if (session('login_fails', 0) > 3) { if (session('login_fails', 0) > 3) {
if (strtolower($request->input('captcha')) != strtolower(session('phrase'))) $this->validate($request, ['captcha' => 'required|captcha']);
return json(trans('auth.validation.captcha'), 1);
} }
if (! $user) { if (! $user) {

22
tests/AuthControllerTest.php
View File

@ -93,19 +93,15 @@ class AuthControllerTest extends TestCase
$this->flushSession(); $this->flushSession();
// Should check captcha if there are too many fails // Should check captcha if there are too many fails
$this->withSession( $this->withSession(['login_fails' => 4])
[ ->postJson(
'login_fails' => 4 '/auth/login', [
] 'identification' => $user->email,
)->postJson( 'password' => '12345678',
'/auth/login', [ ])->assertJson([
'identification' => $user->email, 'errno' => 1,
'password' => '12345678', 'msg' => trans('validation.required', ['attribute' => 'captcha'])
'captcha' => 'b' ]);
])->assertJson([
'errno' => 1,
'msg' => trans('auth.validation.captcha')
]);
$this->flushSession(); $this->flushSession();