Bug（バグ） #3504
closed
新規登録時に「秘密の質問への答え」が空欄のままでも登録できてしまう
100%
Description
Files
Updated by Akihiro KOBAYASHI over 10 years ago
- Status changed from New（新規） to Accepted（着手）
- Assignee set to Akihiro KOBAYASHI
パスワード再発行時にできるということから探った結果、https://trac.openpne.jp/browser/OpenPNE3/plugins/opAuthMailAddressPlugin/trunk/lib/form/opAuthMailAddressPasswordRecoveryForm.class.php?rev=13195
参考になりそうです(特にpublic function checkSecretQuestion)
Updated by Akihiro KOBAYASHI over 10 years ago
secret_answerに初期値が勝手に入っているため、IsRequiredがtrueになっているにもかかわらず通ってしまうようです
http://ja.wikipedia.org/wiki/MD5
入力が0バイト：d41d8cd98f00b204e9800998ecf8427e
Updated by Akihiro KOBAYASHI over 10 years ago
- Related to Bug（バグ） #2972: メンバー登録で「秘密の質問の答え」を入力しないでメンバー登録ができてしまう added
Updated by Akihiro KOBAYASHI over 10 years ago
- Related to Bug（バグ） #2944: 「秘密の質問への答え」に半角及び全角スペースが登録できてしまう added
Updated by Akihiro KOBAYASHI over 10 years ago
プルリクエストしました
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'] != '')
に変更しました
Updated by Akihiro KOBAYASHI over 10 years ago
https://github.com/openpne/OpenPNE3/pull/190
プルリクエストしました
is_null()では空文字列を検知することはできないので、代わりにemptyを用いました
また、行の長さを抑えました
Updated by Akihiro KOBAYASHI over 10 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
Updated by 誠二 天重 over 10 years ago
- Related to Bug（バグ） #2598: 他の認証プラグインで「秘密の質問」の項目が表示されてしまう added
Updated by isao sano about 10 years ago
- Copied to Backport（バックポート） #3719: 新規登録時に「秘密の質問への答え」が空欄のままでも登録できてしまう added
Updated by isao sano about 10 years ago
- Copied to Backport（バックポート） #3720: 新規登録時に「秘密の質問への答え」が空欄のままでも登録できてしまう added
Updated by isao sano about 10 years ago
- Target version changed from OpenPNE 3.8.x to OpenPNE 3.9.0-old
Updated by isao sano about 10 years ago
3.6系ではopAuthMailAddressPlugin が同梱されていないため、プロジェクトをopAuthMailAddressPlugin に移動しました。試験時にご注意ください。
https://redmine.openpne.jp/issues/3719
Updated by Akihiro KOBAYASHI about 10 years ago
プルリクエスト先変更のため
https://github.com/openpne/OpenPNE3/pull/190 を一旦クローズ、
新たに https://github.com/openpne/OpenPNE3/pull/228 を追加
Updated by Youichi Kimura over 8 years ago
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
if (!empty($request['member_config']['secret_answer']))
上記の条件式は $request['member_config']['secret_answer'] (秘密の質問への答え) に 0 と入力された場合も false を返すため適切ではありません。
参照: PHP: empty - Manual
Updated by isao sano about 8 years ago
- Status changed from Rejected（差し戻し） to Accepted（着手）
- % Done changed from 50 to 0
Updated by isao sano about 8 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
#note-14 でプルリクエストされたリポジトリが現在では存在しないため、新しくプルリクエストを作成しました。
https://github.com/openpne/OpenPNE3/pull/395
Updated by Youichi Kimura almost 8 years ago
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
下記 Pull Request 内に記載した箇所について修正をお願いします。
https://github.com/openpne/OpenPNE3/pull/395#pullrequestreview-22708237
Updated by kaoru n almost 8 years ago
- Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
対象バージョン変更のため、修正内容の確認を行います。
Updated by isao sano almost 8 years ago
- Status changed from Rejected（差し戻し） to Accepted（着手）
- % Done changed from 50 to 0
Updated by isao sano almost 8 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
#note-20 を含めて master ブランチにプルリクエストしました。
https://github.com/openpne/OpenPNE3/pull/458
にてプルリクエストを行いました。
Updated by Youichi Kimura over 7 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
レビューOKです
Updated by Youichi Kimura over 7 years ago
- Related to deleted (Bug（バグ） #2972: メンバー登録で「秘密の質問の答え」を入力しないでメンバー登録ができてしまう)
Updated by Youichi Kimura over 7 years ago
- Has duplicate Bug（バグ） #2972: メンバー登録で「秘密の質問の答え」を入力しないでメンバー登録ができてしまう added
Updated by kaoru n over 6 years ago
- Target version changed from OpenPNE 3.9.0 to OpenPNE-3.9.3
Updated by Chiharu Nakajima over 6 years ago
- Status changed from Pending Testing（テスト待ち） to Pending Merge（マージ待ち）
- % Done changed from 70 to 80
動作確認OKです
Updated by kaoru n over 6 years ago
- Status changed from Pending Merge（マージ待ち） to Fixed（完了）
- % Done changed from 80 to 100
マージしました