Как проверить правильность текущего пароля в Laravel 5.8 - PullRequest
1 голос
/ 15 апреля 2019

Я пытаюсь проверить, совпадает ли текущий пароль с БД.

мой упрощенный контроллер:

class ChangePasswordController extends Controller
{
    public function update(Request $request, User $user)
    {
        $this->validate($request, [
            'current_password' => ['required', new CurrentPassword()],
            'password' => 'required|string|min:6|confirmed'
        ]);

        $user->update([
            'password' => bcrypt($request->password)
        ]);

    }
}

В моем собственном правиле CurrentPassword я проверяю хеш следующим образом:

class CurrentPassword implements Rule
{

    public function passes($attribute, $value)
    {
        $check = Hash::check($value, auth()->user()->password);
        dump($check);
        return $check;
    }


    public function message()
    {
        return 'Current password is incorrect.';
    }
}

и мой тест для пользовательского правила:

/** @test */
public function an_authenticated_user_may_change_own_password()
{
    $this->withoutExceptionHandling();

    $user = factory(User::class)->create([
        'password' => '1234'
    ]);

    $this->actingAs($user)->patch("/profile/{$user->id}/password", [
        'current_password' => '1234',
        'password' => 'mynewpassword',
        'password_confirmation' => 'mynewpassword'
    ]);

    $this->assertTrue(Hash::check('mynewpassword', $user->fresh()->password));
}    

к сожалению, я получаю сообщение об ошибке:

1) Тесты \ Feature \ UpdatePasswordTest :: an_authenticated_user_may_change_own_password Осветить \ Validation \ ValidationException: указанные данные были недействительными.

Я не понимаю, почему это радует. Мой dump($check); возвращает false, когда я запускаю этот тест. Мое значение $ равно 1234, а auth () -> user () -> password возвращает также «1234». Может быть, у кого-то есть представление о том, что я делаю неправильно.

этот тест становится зеленым:

 /** @test */
    public function current_password_must_be_valid()
    {
        $user = factory(User::class)->create([
            'password' => '1234'
        ]);

        $this->actingAs($user)->patch("/profile/{$user->id}/password", [
            'current_password' => '12345',
            'password' => 'mynewpassword',
            'password_confirmation' => 'mynewpassword'
        ])->assertSessionHasErrors('current_password');

    }

1 Ответ

1 голос
/ 15 апреля 2019

Вы должны также хэшировать свой пароль на своем заводе, в противном случае Eloquent будет хранить его в открытом виде (поэтому auth()->user()->password возвращает '1234')

public function current_password_must_be_valid()
{
    $user = factory(User::class)->create([
        'password' => Hash::make('1234'); // remember to import the Hash facade
    ]);

    ...
}
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...