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

Rate limit includes successful logins #14507

Open
1 task
zyoshoka opened this issue Sep 6, 2024 · 1 comment
Open
1 task

Rate limit includes successful logins #14507

zyoshoka opened this issue Sep 6, 2024 · 1 comment
Labels
⚠️bug? This might be a bug packages/backend Server side specific issue/PR

Comments

@zyoshoka
Copy link
Contributor

zyoshoka commented Sep 6, 2024

💡 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
@zyoshoka zyoshoka added the ⚠️bug? This might be a bug label Sep 6, 2024
@KisaragiEffective KisaragiEffective added the packages/backend Server side specific issue/PR label Sep 6, 2024
@KisaragiEffective
Copy link
Collaborator

(個人的には成功もレートリミットに含まれるべきではないかという思いもあります。ユースケースが思いつきませんでした)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug packages/backend Server side specific issue/PR
Projects
None yet
Development

No branches or pull requests

2 participants