LaravelでHash::checkがうまくいきません。

Question

Laravel6で以下のようなユーザーのパスワード変更機能を作っています。

以下のように、ユーザー登録のときにHash::makeでパスワードをハッシュ化してデータベースに保存し、パスワード変更の際に現在のパスワードの確認のためにHash::checkを使って判定をしています。

↓RegisterController.php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\User;
use Illuminate\Support\Facades\Hash;

class RegisterController extends Controller
{
    // ユーザー登録の処理
    public function register(Request $request)
    {
        $form = $request->
                validate(User::$registerRules, User::$registerValMessages);

        User::create([
            'name' => $request->name,
            'email' => $request->email,
            'password' => Hash::make($request->password),
        ]);
    }
}

↓UserController.php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use Illuminate\Validation\ValidationException;
use Illuminate\Support\Facades\Hash;
use App\User;

class UserController extends Controller
{
    // ユーザーのパスワードの更新
    public function password(Request $request)
    {
        $request->validate(User::$passwordRules, User::$passwordValMessages);

        $user = User::find($request->id);

        print_r($user->password);
        print_r($request->currentPassword);

        // ここでHash::checkを使って現在のパスワードを確認！！
        if (Hash::check($request->currentPassword, $user->password)) {
            $user->password = Hash::make($request->newPassword);
        }
        throw ValidationException::withMessages([
            'msg' => '現在のパスワードが間違っています！',
        ]);

        $user->save();
    }
}

しかしなぜだかこのコードではHash::checkを通過できず、ValidationExceptionのエラーメッセージが投げられてしまいます。

上のようにHash::checkを行う直前にprint_rで、

DBのユーザーのパスワード：$user->password
入力した現在のパスワード：$request->currentPassword

をチェックしたのですが、前者はハッシュ化された現在のパスワード、後者はリクエストで送られてきた平文文字列のパスワードをきちんと取得できました。

何度も試したので、パスワードそのものが間違っているということはまずないと思います。

Hash::checkがうまくいっていない要因として何が考えられるでしょうか？

Answer 1

        // ここでHash::checkを使って現在のパスワードを確認！！
        if (Hash::check($request->currentPassword, $user->password)) {
            $user->password = Hash::make($request->newPassword);
        }

//問答無用でthrowしている
        throw ValidationException::withMessages([
            'msg' => '現在のパスワードが間違っています！',
        ]);

throwをelseで括れば良いでしょう

Comments

[izaya]

初歩的なミスでしたね。。
どうもありがとうございました。