-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Send "429 Too Many Requests" in case of brute force protection #22280
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice 👍 Also the move of the count functionality into the SQL statement is a good catch 🚀
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
4bdae65
to
6f5f71d
Compare
Rebased and fixed the tests |
Signed-off-by: Joas Schilling <coding@schilljs.com>
Not sure whats going on with the psalm thing and how to fix this. @MorrisJobke |
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 31983: failuremysql8.0-php7.2Show full log
|
The fix for the static code analysis is in #22304 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From me again 👍
No description provided.