プロジェクト

全般

プロフィール

Bug(バグ) #3504

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

kaoru n10年以上前に追加. 5年以上前に更新.

ステータス:
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

関係しているリビジョン

リビジョン be7d3d28 (差分)
isao sanoほぼ7年前に追加

add a requirement not to register the secret password with null character (fixes #3504)

リビジョン 73d79a4d (差分)
isao sanoほぼ7年前に追加

Change to not allow blanks (refs #3504)

リビジョン 8c375de9
kaoru n5年以上前に追加

Merge pull request #458 from isaosano/t3504

add a requirement not to register the secret password with null character (fixes #3504)

履歴

#1 kaoru n10年以上前に更新

  • 説明 を更新 (diff)

#2 Akihiro KOBAYASHIほぼ10年前に更新

  • ステータス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 KOBAYASHIほぼ10年前に更新

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

#4 Akihiro KOBAYASHIほぼ10年前に更新

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

#5 Akihiro KOBAYASHIほぼ10年前に更新

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

#6 Akihiro KOBAYASHIほぼ10年前に更新

プルリクエストしました
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 KOBAYASHIほぼ10年前に更新

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

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

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

#8 Akihiro KOBAYASHI9年以上前に更新

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

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

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

#10 isao sano9年以上前に更新

#11 isao sano9年以上前に更新

#12 isao sano9年以上前に更新

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

#13 isao sano9年以上前に更新

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

#14 Akihiro KOBAYASHI9年以上前に更新

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

#15 kaoru nほぼ9年前に更新

  • 担当者 を削除 (Akihiro KOBAYASHI)

#16 kaoru n7年以上前に更新

  • 担当者isao sano にセット

#17 Youichi Kimura7年以上前に更新

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

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

#18 isao sano7年以上前に更新

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

#19 isao sano7年以上前に更新

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

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

#20 Youichi Kimura約7年前に更新

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

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

#21 kaoru n約7年前に更新

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

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

#22 isao sanoほぼ7年前に更新

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

#23 isao sanoほぼ7年前に更新

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

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

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

#24 Youichi Kimura6年以上前に更新

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

レビューOKです

#25 Youichi Kimura6年以上前に更新

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

#26 Youichi Kimura6年以上前に更新

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

#27 kaoru n5年以上前に更新

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

#28 Chiharu Nakajima5年以上前に更新

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

動作確認OKです

#29 kaoru n5年以上前に更新

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

マージしました

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