Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

会員登録済みのメールアドレスかどうか判別が可能 #5279

Open
taguchimasahiro opened this issue Jan 26, 2022 · 5 comments
Labels
security security
Milestone

Comments

@taguchimasahiro
Copy link
Contributor

taguchimasahiro commented Jan 26, 2022

概要(Overview)

新規会員登録画面で、メールアドレスを入力した際に表示されるエラーメッセージから、すでに会員登録済みのメールアドレスかどうかが判別できます。

期待する内容(Expect)

メールアドレスが会員登録済みかどうかに関わらない挙動。

再現手順(Procedure)

新規会員登録で、すでに会員登録されているメールアドレスを利用して登録しようとすると、「このメールアドレスは利用できません」とエラーメッセージが表示される。

なお、2022/01/25時点にて、同様の再現手順でShopify,MakeShop,futureshop,ebisumartで確認したところ、以下のエラーメッセージが表示され、当Issueと同様に会員登録済みのメールアドレスかどうか判別が可能でした。
Shopify「このメールアドレスは既にアカウントと紐付けられています。」
MakeShop「入力されたE-mailは既にログインIDで使用されています。」
futureshop「このメールアドレスはすでに登録されています。」
ebisumart「入力されたメールアドレスはすでに登録されています。」

環境 (environment)

  • EC-CUBE: 4.1.0
  • PHP: 7.x.x
  • DB:
    • PostgreSQL x.x.x
    • MySQL x.x.x

関連情報 (Ref)

@taguchimasahiro taguchimasahiro added this to the 4.1.x milestone Jan 26, 2022
@chihiro-adachi chihiro-adachi modified the milestones: 4.1.x, 4.2.0 May 30, 2022
@chihiro-adachi
Copy link
Contributor

chihiro-adachi commented Jun 7, 2022

以下の実装案で検討しています。

  • メールアドレスの重複チェックを外し、登録済メールアドレスでも登録できるようにする
  • 登録処理実行時、すでにメールアドレスが登録されている場合は
    • 仮会員で登録されている場合
      • 登録処理をスキップ
      • 完了メールにで「仮会員で登録されています。登録を完了してください。」の旨のメールを送付する
    • 本会員で登録されている場合
      • 登録処理をスキップ
      • 完了メールにで「すでに登録完了しています。マイページからログインしてください」の旨のメールを送付する

@chihiro-adachi
Copy link
Contributor

#5391 で対応

@chihiro-adachi
Copy link
Contributor

#5391 にて修正を実施していましたが、以下の理由により修正を取り下げます。
仕様については改めて検討となります。

  • マイページ>会員情報編集画面においてもメールアドレスの重複チェックが存在する
    • 会員登録のフロー変更だけでは修正が十分でない
    • メールアドレスの変更画面を別画面に切り出すなど、影響の大きな改修が必要
  • 仕様が理解しづらい、複雑すぎる
    • 仮会員の設定 x 会員のステータスの組み合わせで処理が異なり、ユーザ・店舗主・開発者それぞれに負担となる

提供できるメリットよりデメリットが大きいと判断し、修正をrevertします。

@shinya
Copy link
Contributor

shinya commented Oct 25, 2022

実装の検討は引き続き進めたいが、決めないといけない仕様の部分が多岐にわたるので、4.2.1のスコープに含めるかどうかはもう少し議論が必要。

@xuelian311 xuelian311 modified the milestones: 4.x, 4.2.2 Apr 27, 2023
@xuelian311
Copy link
Contributor

4.2.2の対象として検討を始めたいので、リファインメントからやり直します。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
security security
Projects
None yet
Development

No branches or pull requests

4 participants