Project

General

Profile

Backport(バックポート) #4017

Bug(バグ) #4012: メールアドレス設定から意図的に他のメンバーと同じメールアドレスを設定できる

メールアドレス設定から意図的に他のメンバーと同じメールアドレスを設定できる

Added by Youichi Kimura over 2 years ago. Updated 4 months ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2016-09-23
Due date:
% Done:

100%


Description

Overview (概要)

再現手順:

  1. メンバーA でログインし、PCメールアドレス設定 (/member/config?category=pcAddress) を開く
  2. PCメールアドレスに hoge@example.com を入力して「送信」ボタンをクリック
    • hoge@example.com 宛に確認メールが届くが、この時点ではメールに記載されている URL を開かない
  3. メンバーB でログインし、PCメールアドレス設定 (/member/config?category=pcAddress) を開く
  4. PCメールアドレスに hoge@example.com を入力して「送信」ボタンをクリック
  5. メンバーA によって送られた「メールアドレス変更ページのお知らせ」のメールに記載されている URL を開く
  6. メンバーA のパスワードを入力して「送信」ボタンをクリック
  7. メンバーB によって送られた「メールアドレス変更ページのお知らせ」のメールに記載されている URL を開く
  8. メンバーB のパスワードを入力して「送信」ボタンをクリック
  9. メンバーA, メンバーB のPCメールアドレスが同じ hoge@example.com に設定された状態になる

Causes (原因)

lib/config/config/member_config.yml では pc_address および mobile_address に対して IsUnique: true が指定されており、実際に「PCメールアドレス設定」のフォームでは MemberConfigForm::isUnique() によって重複チェックが行われている。
しかし、メールアドレス変更については、変更後のメールアドレス宛に /member/configComplete?token=*** のURLをメールで送った上で、確認が完了するまでは pc_address を変更しないという例外的な対応を取っている。そのため、MemberConfigForm のバリデーション時には重複しなかったが /member/configComplete の時点で重複するような場合には対処できていなかった。

Way to fix (修正内容)

opMemberActions::executeConfigComplete() において、pc_address_pre または mobile_address_pre を pc_address または mobile_address に変更する直前にも重複チェックを行う。これにより、上記の再現手順でいう手順 8 の段階でエラーが発生するようになる。

なお、手順 4 の段階で pc_address_pre が重複するためこの時点でエラーを発生させるといった修正も考えられるが、これをエラーとしてしまうと悪意のあるメンバーが任意のメールアドレスを確認が未完了のまま放置することによって他のメンバーがそのメールアドレスを使用できない状態に出来てしまうため、pc_address_pre に対する重複チェックは行わない。


Related issues

Related to OpenPNE 3 - Backport(バックポート) #4015: メールアドレス変更処理が完了しても、仮登録用のデータ (***_pre, ***_token) が MemberConfig に残ってしまい、削除されない Fixed(完了) 2012-06-14

History

#1 Updated by Youichi Kimura over 2 years ago

  • Related to Bug(バグ) #4012: メールアドレス設定から意図的に他のメンバーと同じメールアドレスを設定できる added

#2 Updated by Youichi Kimura over 2 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

下記の Pull Request にて修正しました (#4015, #4017 の修正を含んでいます)
https://github.com/openpne/OpenPNE3/pull/374

#4 Updated by Shinichi Urabe over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

バグチケットと同様の理由で差し戻し

#5 Updated by isao sano about 2 years ago

  • Related to Backport(バックポート) #4015: メールアドレス変更処理が完了しても、仮登録用のデータ (***_pre, ***_token) が MemberConfig に残ってしまい、削除されない added

#6 Updated by kaoru nishizoe over 1 year ago

  • Parent task set to #4012

#7 Updated by isao sano about 1 year ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#8 Updated by isao sano about 1 year ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#9 Updated by Rimpei Ogawa 5 months ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

親チケットでの指摘がこちらでは修正されていません
https://redmine.openpne.jp/issues/4012#note-13

#10 Updated by kaoru nishizoe 5 months ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

#11 Updated by Rimpei Ogawa 5 months ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです。

#12 Updated by kaoru nishizoe 5 months ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.33

#13 Updated by Chiharu Nakajima 4 months ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

動作確認OKです

#14 Updated by kaoru nishizoe 4 months ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF