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

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

ただいまの
回答率

89.20%

パスワードリセットのパスワード上書きについて

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,841

TadashiOsanai

score 10

こんにちは。
現在、ログイン機能を実装しております。
そこで、パスワード忘れた方へのパスワードリセットページを作成しております。
下記のソースで上手く動きません。

PHPの勉強中で、ログインのINSERTとちょっと変更したら登録できるのかなと試行錯誤しております。そもそものコードがおかしいのかわからないく、指摘をいただけないでしょうか?

<?php


include ('config.php');
include ('function.php');

$mail = $_GET['mail'];

$userID = UserID($mail); 

$verifytoken = verifytoken($userID, $token);


//パスワードのハッシュ化


if(isset($_POST['submit'])){

$new_password = $_POST['new_password'];
    $password_hash =  password_hash($_SESSION['new_password'], PASSWORD_DEFAULT);
    $password_hash =  password_hash($_SESSION['retype_password'], PASSWORD_DEFAULT);

    if($new_password == $retype_password)
    {

   try{
      $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

      //トランザクション開始
      $dbh->beginTransaction();

      $statement = $dbh->prepare("UPDATE member SET password = '$new_password' WHERE id = $userID");
      $statement->bindValue(':new_password_hash', $new_password_hash, PDO::PARAM_STR);
      $statement->execute();

      if($update_password){
                $statement = $dbh->prepare("UPDATE recovery_keys SET valid = 0 WHERE userID = $userID AND token ='$token'");
                $msg = 'Your password has changed successfully. Please login with your new passowrd.';
                $msgclass = 'bg-success';
        }
        }else
    {
         $msg = "パスワードが一致しません。";
         $msgclass = 'bg-danger';
    }

      // トランザクション完了(コミット)
      $dbh->commit();

      }catch (PDOException $e){
        print('Error:'.$e->getMessage());
        die();
    }

}


?>
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="utf-8">
<meta http-equiv="X-UA-Compatible" content="IE=edge">
<meta name="viewport" content="width=device-width, initial-scale=1">
<title>パスワードリセット</title>
<!-- Bootstrap -->
<link href="css/bootstrap.css" rel="stylesheet">
</head>
<body>
<div class="container">
  <div class="row">
    <?php if($verifytoken == 1) { ?>
    <div class="col-lg-4 col-lg-offset-4">
      <form class="form-horizontal" role="form" method="post">
        <h2>Reset Your Password</h2>
        <?php if(isset($msg)) { ?>
        <div class="<?php echo $msgclass; ?>" style="padding:5px;"><?php echo $msg; ?></div>
        <?php } ?>
        <div class="row">
          <div class="col-lg-12">
            <label class="control-label">新しいパスワード</label>
          </div>
        </div>
        <div class="row">
          <div class="col-lg-12">
            <input class="form-control" name="new_password" type="password" placeholder="New Password" required>
          </div>
        </div>
        <div class="row">
          <div class="col-lg-12">
            <label class="control-label">再入力</label>
          </div>
        </div>
        <div class="row">
          <div class="col-lg-12">
            <input class="form-control" name="retype_password" type="password" placeholder="Re-type New Password" required>
          </div>
        </div>
        <div class="row">
          <div class="col-lg-12">
            <button class="btn btn-success btn-block" name="submit" style="margin-top:8px;">Submit</button>
          </div>
        </div>
      </form>
    </div>
    <?php }else {?>
    <div class="col-lg-4 col-lg-offset-4">
      <h2>Invalid or Broken Token</h2>
      <p>Opps! The link you have come with is maybe broken or already used. Please make sure that you copied the link correctly or request another token from <a href="index.php">here</a>.</p>
    </div>
    <?php }?>
  </div>
</div>
</body>
</html>

参考にしたコードは下記になります。
こちら、パスワードの暗号がMD5で古いようなので、PDOの
password_hashに書き換えたいと思い、挑戦して言います。

if(isset($_POST['submit']))
{
    $new_password = $_POST['new_password'];
    $new_password = md5($new_password);
    $retype_password = $_POST['retype_password'];
    $retype_password = md5($retype_password);

    if($new_password == $retype_password)
    {
        $update_password = mysqli_query($db, "UPDATE member SET password = '$new_password' WHERE id = $userID");
        if($update_password)
        {
                mysqli_query($db, "UPDATE recovery_keys SET valid = 0 WHERE userID = $userID AND token ='$token'");
                $msg = 'Your password has changed successfully. Please login with your new passowrd.';
                $msgclass = 'bg-success';
        }
    }else
    {
         $msg = "Password doesn't match";
         $msgclass = 'bg-danger';
    }

}

もしもっといい方法があるようでしたら、そちらもヒントを教えていただけないでしょうか。
よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • y_waiwai

    2018/07/30 16:56

    上手く動かないとは、どういうふうに動くんでしょうか。詳しく説明してください

    キャンセル

  • TadashiOsanai

    2018/07/30 17:35

    ご連絡ありがとうございます。画面が表示されません。

    キャンセル

  • m.ts10806

    2018/07/30 21:18

    質問は編集できますので適宜調整してください。

    キャンセル

  • m.ts10806

    2018/07/30 21:20

    エラー表示はOnにして確認してください。何も表示されない=処理が中断するようなエラーが起きているのにエラー表示がOnになっていないため何も起きていないように見える と思われます。https://qiita.com/Morio/items/d83092caf3a32f94b506

    キャンセル

回答 1

+4

$password_hash =  password_hash($_SESSION['new_password'], PASSWORD_DEFAULT);
$password_hash =  password_hash($_SESSION['retype_password'], PASSWORD_DEFAULT);
どちらも$password_hashに入っています。

更に$retype_password$new_password_hash$update_passwordなど定義されてない変数が見受けられます。

更に更に$dbh->prepare内で変数を直入れしているのでプリペアドステートメント出来てません。

参考にしたコードは見た感じ問題無いと思うので、自分のコードを理解しながら全体的に見直した方がいいと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/07/30 17:41

    ありがとうございます。
    $statement = $dbh->prepare("INSERT INTO member (account,mail,password) VALUES (:account,:mail,:password_hash)");
    ログイン画面のIDとメール、パスワードを作成しましたが、こちらの形に近いのでしょうか?

    また、md5($new_password);の部分を password_hash($_SESSION['new_password'], PASSWORD_DEFAULT);にする場合、全体を見直したほうが良いのでしょうか?
    それとも少し修正すれば上手くいくのでしょうか?
    その辺も、ヒントをいただければと思います。

    キャンセル

  • 2018/07/30 18:11

    近いも何も、そうしないとSQLインジェクションの危険性から逃れられません。
    以下のページが詳しいのでご覧ください。
    https://qiita.com/mpyw/items/b00b72c5c95aac573b71

    password_hashについては、md5(〜)をpassword_hash(〜)に置き換えればそれでOKです。
    (認証チェックの方法は以下参照)
    https://qiita.com/rana_kualu/items/3ef57485be1103362f56

    「上手くいくかどうか?」と思った際には、まずは「php pdo」とか「php password_hash」でググれば例文や解説がいっぱい出て来ます。
    トライ&エラーしつつ理解を進めてみてください(1つの題材でも色々なサイトを見てみるとよいです)。

    そして画面が表示されないとのことですが、エラーも非表示設定になっていると思われるので
    <?php の次の行あたりにでも
    ini_set( 'display_errors', 1 );
    と書いておくとエラー文が画面に表示されるようになります。(リリース時はこの行を消してください)
    何言ってるのか分からないエラーもあるとは思いますが、文面をそのままググれば解決の助けになります。
    https://qiita.com/shotets/items/3c95aef631b2c5eadae5

    キャンセル

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

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