misskey icon indicating copy to clipboard operation
misskey copied to clipboard

fix: 登録メール送信時に重複確認を行う

Open Ry0taK opened this issue 3 years ago • 3 comments

What

  • signupエンドポイントにおいて、メールアドレスの検証が走る場合にユーザー名の重複確認が行われていなかった問題を修正

Why

  • 既に登録が完了しているユーザー名で登録メールを送れてしまい、ユーザーの混乱を招くため

Additional info (optional)

  • partially fixes #10076
  • 既に登録プロセスが完了している場合に登録が出来ないようになりますが、同時に同じユーザー名で登録プロセスを開始した場合、上記Issueで説明されている問題が依然として発生します。
    • Pendingになっているユーザーのリポジトリにおいても重複確認を行うべきか検討しましたが、メールアドレスを間違えた場合などに再度登録したいというケースがあることを想定して一旦はそのままにしています。
    • そのため、メールアドレス認証ページにおいて別途エラーハンドリングを行い、適切なメッセージを表示するべきかもしれません
  • メール登録を必須にしているインスタンスにおいて、以下の状況において動作確認済み
    1. ユーザー名が重複している場合、400が帰ることを確認
    2. ユーザー名が重複していない場合、204が帰りメールが飛んでくることを確認

Ry0taK avatar Mar 06 '23 06:03 Ry0taK

Codecov Report

Merging #10231 (3f403e2) into develop (f5b63d8) will decrease coverage by 0.01%. The diff coverage is 30.00%.

@@             Coverage Diff             @@
##           develop   #10231      +/-   ##
===========================================
- Coverage    69.23%   69.22%   -0.01%     
===========================================
  Files          707      707              
  Lines        65407    65420      +13     
  Branches      5213     5215       +2     
===========================================
+ Hits         45283    45290       +7     
- Misses       20124    20130       +6     
Impacted Files Coverage Δ
...ackages/backend/src/server/api/SignupApiService.ts 50.23% <26.31%> (-1.26%) :arrow_down:
packages/backend/src/misc/correct-filename.ts 100.00% <100.00%> (ø)
packages/backend/src/core/NotificationService.ts 100.00% <0.00%> (+4.28%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Mar 06 '23 10:03 codecov[bot]

これクライアント側の表示ってどうなります?

rinsuki avatar Mar 06 '23 12:03 rinsuki

これクライアント側の表示ってどうなります?

ボタンは押せますが、押しても何も反応しない状態になります (ボタンを押せなくするのは #10238 で対応しています)

Ry0taK avatar Mar 06 '23 13:03 Ry0taK

🙏🏻

syuilo avatar Mar 09 '23 03:03 syuilo