zhangyuheng/blessing-skin-server
1
0
Fork 0
forked from mirror/blessing-skin-server
Code Pull Requests Activity

Fix validating captcha

Browse Source
This commit is contained in:
Pig Fang 2018-08-12 16:00:21 +08:00
parent b343707978
commit b3876468b2
2 changed files with 4 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -140,8 +140,9 @@ class AuthController extends Controller
public function handleForgot(Request $request, UserRepository $users)
{
if (! $this->checkCaptcha($request))
return json(trans('auth.validation.captcha'), 1);
$this->validate($request, [
'captcha' => 'required'.(app()->environment('testing') ? '' : '|captcha')
]);
if (config('mail.driver') == "")
return json(trans('auth.forgot.close'), 1);

10
tests/AuthControllerTest.php
View File

@ -372,17 +372,9 @@ class AuthControllerTest extends TestCase
{
Mail::fake();
// Should return a warning if `captcha` is wrong
$this->withSession(['phrase' => 'a'])->postJson('/auth/forgot', [
'captcha' => 'b'
])->assertJson([
'errno' => 1,
'msg' => trans('auth.validation.captcha')
]);
// Should be forbidden if "forgot password" is closed
config(['mail.driver' => '']);
$this->withSession(['phrase' => 'a'])->postJson('/auth/forgot', [
$this->postJson('/auth/forgot', [
'captcha' => 'a'
])->assertJson([
'errno' => 1,