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

Add: #348 新規登録の上限人数 #527

Merged
merged 11 commits into from
Feb 12, 2024

Conversation

kmycode
Copy link
Owner

@kmycode kmycode commented Feb 7, 2024

Closes #348, #349

  • 制限に達しているかの確認は、新規登録画面で登録した時(メール認証前)/メール認証時、の2箇所で行います
    • ユーザーから見れば『メール認証は来たのに承認できない』という状況が起こることが想定されます
    • 大量のメール認証を発行して他のユーザーの登録を妨害する攻撃のため、『認証済ユーザー』でカウントします
    • 大量のメール認証を制限を超えるほど大量に発行した後でそれらを一気に認証する攻撃のため、メール認証時にも確認を行います
    • メール認証前の確認は、単純にユーザーの利便性が目的です
  • 招待されている場合、制限を貫通します
  • 制限に到達している場合でも、tootctl accounts createは可能です

#348 人数上限

  • 内部処理
  • 設定画面(現在の登録人数も表示のこと)
  • テストコード作成
  • 開発環境で動作確認

#349 人数上限(1日あたり)

  • 内部処理
  • 日付変わったらキャッシュをリセットする処理
  • 設定画面(UTC基準であること/現在の登録人数も表示のこと)
  • テストコード作成
  • 開発環境で動作確認

登録可能時間帯

  • 内部処理
  • 設定画面(UTC基準であることを表示のこと)
  • テストコード作成
  • Secondary時間帯のテストコード作成
  • 開発環境で動作確認

@kmycode kmycode added this to the kb11.0 milestone Feb 7, 2024
@kmycode kmycode linked an issue Feb 7, 2024 that may be closed by this pull request
@kmycode kmycode marked this pull request as ready for review February 12, 2024 13:05
@kmycode kmycode merged commit e317ede into kb_development Feb 12, 2024
52 checks passed
@kmycode kmycode deleted the kbtopic-348-registrations-limit branch February 12, 2024 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

登録人数上限を設定(1日あたり) 登録人数上限を設定
1 participant