質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.98%

  • CakePHP

    2129questions

    CakePHPは、PHPで書かれたWebアプリケーション開発用のフレームワークです。 Ruby on Railsの考え方を多く取り入れており、Railsの高速性とPHPの機動性を兼ね備えています。 MVCやORMなどを「規約優先の考え方」で利用するため、コードを書く手間を省くことができます。 外部のライブラリに依存しないので、単体での利用が可能です。

cakephp3でパスワードリマインダー実装

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 459

lovelydai

score 20

こんにちは、Cakephp3でログインできるシステムを作っています。

ログインしていない状態で、顧客番号とログインIDと現在パスワード、新しいパスワードを入れることでパスワードを更新させる機能を実装したいです。利用者登録するとランダムでパスワードが発行され、メールでお知らせする仕組みなのでログインなしでもパスワードが変更できるようにしたいです。

認証はCakephp3のAuthを利用して作り、問題なく動いています。
そこで、ページからPOSTで入力されたデータを利用し、DBに暗号化されたパスワードと一致するかを比較し、一致した場合は利用者が入力した新パスワードで更新する機能を作りたくて以下のようにコードを作ってみましたが、エラーが出てしまいました。

    //UsersController.phpの中

public function change()
    {

      if ($this->request->is('post'))
      {

        $salt_key = '//システムの基本値64Bitのコードを入れました。';

        $customer_no = $this->request->data['customer_no'];
        $user_id = $this->request->data['user_id'];
        $password_now = $this->request->data['password_now'];
        $password_new = $this->request->data['password_new'];
        $password_check = $this->request->data['password_check'];

        $user = $this->Users->find()->where(['customer_no' => $customer_no])
                                    ->andWhere(['user_id' => $user_id]);

        $pwd = $user->password; // エラー: Undefined property: Cake\ORM\Query::$password
        $password = Security::decrypt($pwd,$salt_key);

        if ($password === $password_now)
        {

          if($password_new === $password_check)
          {
            $user->password = Security::hash($password_new,$salt_key);
            $user->modifier = $user_id;
            $this->Users->save($user);

            $this->Flash->success(__('Success to change password, plz login with new password.'));
            return $this->redirect(['action' => 'login']);
          }
          else {
            $this->Flash->error(__('Failed to change password, plz check your informaion.'));
          }

        }
      }
    }

すると、このようなエラーメッセージが画面にでてしまいます。
![イメージ説明](de145520273f932c22f6f414b708244d.png)

$passwordという変数が定義されてないと出るし、Decryptもそれでうまく動いてなさそうです。そもそも、Hash化されてDBにあるパスワードをこのように無理やり読み込んで復号化し、POSTデータと比較してまた暗号化するというやり方が正しいか?という疑問もあります。
また、ControllerとModel(UsersTable.php)に分けてコードを書く必要があるか?と思っていますが…データベースに接続してパスワードの一致を確認する処理は、コントローラーではなくてModelか、Entityですることかな?とかのもやもや感があります。

何かアドバイスや参考できることがあればぜひお願いしたいです。
どうぞよろしくお願いいたします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • mts10806

    2017/09/25 14:20 編集

    そもそも$userは正しく想定の情報が取得できているのでしょうか?他のrequestも同様にご確認ください。

    キャンセル

  • lovelydai

    2017/09/25 15:35

    Queryの実行されてなかったです。ご指摘ありがとうございます。

    キャンセル

回答 2

check解決した方法

0

解決しました。
原因は、ハッシュ化されたものをチェックする場合DefaultPasswordHasherを利用しないといけない、ということでした。具体的には、$hasher->check()で一致しているか確認し、save()で更新することで正常に動くようになりました。

use Cake\Auth\DefaultPasswordHasher; // 追加

class UsersController extends AppController
{
 // コード省略
  public function change()
    {

      if ($this->request->is('post'))
      {

        $user_id = $this->request->data['user_id'];
        $password_now = $this->request->data['password_now'];
    $password_new = $this->request->data['password_new'];

        $query = $this->Users->find()->where(['user_id' => $user_id]);
        $user = $query->first();

        if($user)
        {
          // 利用者情報の収得に成功した場合
          $hasher = new DefaultPasswordHasher();
          $equal_check = $hasher->check($password_now, $user->password);

          if($equal_check)
          {
              $user->password = $password_new;
              if ($this->Users->save($user))
              {
                $this->Flash->success(__('パスワードを成功的に更新しました。新しいログイン情報でログインしてください。'));
                return $this->redirect(['action' => 'login']);
              }
              else
              {
                $this->Flash->error(__('パスワードの更新に失敗しました。ログイン情報を確認しやり直してください。'));
              }
           }
        }
    }
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

        $user = $this->Users->find()->where(['customer_no' => $customer_no])
                                    ->andWhere(['user_id' => $user_id]);


のあとに

        $user = $user->first();


としたら$user->passwordが取得できるようになるかと思われますがどうでしょう?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/25 15:35

    おかげさまで収得できました!ありがとうございます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.98%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • CakePHP

    2129questions

    CakePHPは、PHPで書かれたWebアプリケーション開発用のフレームワークです。 Ruby on Railsの考え方を多く取り入れており、Railsの高速性とPHPの機動性を兼ね備えています。 MVCやORMなどを「規約優先の考え方」で利用するため、コードを書く手間を省くことができます。 外部のライブラリに依存しないので、単体での利用が可能です。