Skip to content

Rate limit includes successful logins #14507

Open

Description

💡 Summary

サインインする際にレートリミットに達すると TOO_MANY_AUTHENTICATION_FAILURES エラーが出ますが、実際には成功したログインもレートリミットの対象になっています。これではエラーの内容と挙動が一致していないように思います。

🥰 Expected Behavior

エラーの内容と挙動が一致する。具体的には以下2つのうちいずれかの仕様になっていることを期待します。

  • エラーメッセージが Too many failed attempts to sign in. Try again later. ではなく Too many attempts to sign in. Try again later. になっている
  • 成功したログインはレートリミットの対象にならない

🤬 Actual Behavior

エラーの内容と挙動が一致していない

📝 Steps to Reproduce

  1. 適当な本番サーバーで同一 IP アドレスから1時間以内にログインに10回成功した上でもう一度ログインしようとする

💻 Frontend Environment

No response

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey: 2024.8.0
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions