プロジェクト

全般

プロフィール

Bug(バグ) #3504

新規登録時に「秘密の質問への答え」が空欄のままでも登録できてしまう

kaoru nishizoe約5年前に追加. 2ヶ月前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2013-11-28
期日:
進捗率:

100%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

説明

現象

新規登録時に、秘密の質問の答えが空欄のままで新規登録ボタンを押下しても、必須入力エラーにならず登録できてしまう。

調査

パスワード再発行時の秘密の質問の答えを入力しなければならない場面では、未入力のまま送信できない。

原因

未調査

修正内容

未調査

スクリーンショット 2014-06-05 18.14.27.png 表示 (73.6 KB) Akihiro KOBAYASHI, 2014-06-05 18:15


子チケット

opAuthMailAddressPlugin - Backport(バックポート) #3719: 新規登録時に「秘密の質問への答え」が空欄のままでも登録できてしまうFixed(完了)isao sano

Backport(バックポート) #3720: 新規登録時に「秘密の質問への答え」が空欄のままでも登録できてしまうFixed(完了)isao sano


関連するチケット

関連している opAuthMailAddressPlugin - Bug(バグ) #2944: 「秘密の質問への答え」に半角及び全角スペースが登録できてしまう New(新規) 2012-04-11 2012-04-20
関連している opAuthMailAddressPlugin - Bug(バグ) #2598: 他の認証プラグインで「秘密の質問」の項目が表示されてしまう Fixed(完了) 2011-11-15
次のチケットが重複 opAuthMailAddressPlugin - Bug(バグ) #2972: メンバー登録で「秘密の質問の答え」を入力しないでメンバー登録ができてしまう Invalid(無効) 2012-04-16

履歴

#1 kaoru nishizoe約5年前に更新

  • 説明 を更新 (diff)

#2 Akihiro KOBAYASHI4年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Akihiro KOBAYASHI にセット

パスワード再発行時にできるということから探った結果、https://trac.openpne.jp/browser/OpenPNE3/plugins/opAuthMailAddressPlugin/trunk/lib/form/opAuthMailAddressPasswordRecoveryForm.class.php?rev=13195
参考になりそうです(特にpublic function checkSecretQuestion)

#3 Akihiro KOBAYASHI4年以上前に更新

secret_answerに初期値が勝手に入っているため、IsRequiredがtrueになっているにもかかわらず通ってしまうようです
http://ja.wikipedia.org/wiki/MD5
入力が0バイト:d41d8cd98f00b204e9800998ecf8427e

#4 Akihiro KOBAYASHI4年以上前に更新

  • 関連している Bug(バグ) #2972: メンバー登録で「秘密の質問の答え」を入力しないでメンバー登録ができてしまう を追加

#5 Akihiro KOBAYASHI4年以上前に更新

  • 関連している Bug(バグ) #2944: 「秘密の質問への答え」に半角及び全角スペースが登録できてしまう を追加

#6 Akihiro KOBAYASHI4年以上前に更新

プルリクエストしました
https://github.com/openpne/OpenPNE3/pull/136

原因はopAuthRegisterFormMailAddress.class.php内
if (isset($request['member_config']['secret_answer']))
で、空文字でもtrueになるためです

よって if (isset($request['member_config']['secret_answer']) && $request['member_config']['secret_answer'] != '')
に変更しました

#7 Akihiro KOBAYASHI4年以上前に更新

https://github.com/openpne/OpenPNE3/pull/190
プルリクエストしました

is_null()では空文字列を検知することはできないので、代わりにemptyを用いました

また、行の長さを抑えました

#8 Akihiro KOBAYASHI4年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

#9 誠二 天重4年以上前に更新

  • 関連している Bug(バグ) #2598: 他の認証プラグインで「秘密の質問」の項目が表示されてしまう を追加

#10 isao sano約4年前に更新

#11 isao sano約4年前に更新

#12 isao sano約4年前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.9.0-old に変更

#13 isao sano約4年前に更新

3.6系ではopAuthMailAddressPlugin が同梱されていないため、プロジェクトをopAuthMailAddressPlugin に移動しました。試験時にご注意ください。
https://redmine.openpne.jp/issues/3719

#14 Akihiro KOBAYASHI約4年前に更新

プルリクエスト先変更のため
https://github.com/openpne/OpenPNE3/pull/190 を一旦クローズ、
新たに https://github.com/openpne/OpenPNE3/pull/228 を追加

#15 kaoru nishizoe3年以上前に更新

  • 担当者 を削除 (Akihiro KOBAYASHI)

#16 kaoru nishizoe2年以上前に更新

  • 担当者isao sano にセット

#17 Youichi Kimura2年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更
if (!empty($request['member_config']['secret_answer']))

上記の条件式は $request['member_config']['secret_answer'] (秘密の質問への答え) に 0 と入力された場合も false を返すため適切ではありません。
参照: PHP: empty - Manual

#18 isao sano約2年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#19 isao sano約2年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

#note-14 でプルリクエストされたリポジトリが現在では存在しないため、新しくプルリクエストを作成しました。
https://github.com/openpne/OpenPNE3/pull/395

#20 Youichi Kimuraほぼ2年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

下記 Pull Request 内に記載した箇所について修正をお願いします。
https://github.com/openpne/OpenPNE3/pull/395#pullrequestreview-22708237

#21 kaoru nishizoe1年以上前に更新

  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

対象バージョン変更のため、修正内容の確認を行います。

#22 isao sano1年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#23 isao sano1年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

#note-20 を含めて master ブランチにプルリクエストしました。

https://github.com/openpne/OpenPNE3/pull/458
にてプルリクエストを行いました。

#24 Youichi Kimura1年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

レビューOKです

#25 Youichi Kimura1年以上前に更新

  • 関連している を削除 (Bug(バグ) #2972: メンバー登録で「秘密の質問の答え」を入力しないでメンバー登録ができてしまう)

#26 Youichi Kimura1年以上前に更新

  • 次のチケットが重複 Bug(バグ) #2972: メンバー登録で「秘密の質問の答え」を入力しないでメンバー登録ができてしまう を追加

#27 kaoru nishizoe3ヶ月前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE-3.9.3 に変更

#28 Chiharu Nakajima3ヶ月前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

動作確認OKです

#29 kaoru nishizoe3ヶ月前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

他の形式にエクスポート: Atom PDF