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.

Sign up for GitHub

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

Jump to bottom

feat(bruteforce): Allow forcing the database throttler #48884

Merged
merged 1 commit into from
Oct 25, 2024
Merged

feat(bruteforce): Allow forcing the database throttler #48884

merged 1 commit into from
Oct 25, 2024

Conversation

nickvergessen
Copy link
Member

Using the database is most likely worse for performance, but makes investigating issues a lot easier as it's possible to look directly at the table to see all logged remote addresses and actions.

Checklist

@nickvergessen
feat(bruteforce): Allow forcing the database throttler
6854af0 
Using the database is most likely worse for performance, but makes investigating
issues a lot easier as it's possible to look directly at the table to see all
logged remote addresses and actions.

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the Nextcloud 31 milestone Oct 24, 2024
@nickvergessen nickvergessen requested review from SystemKeeper and a team October 24, 2024 20:06
@nickvergessen nickvergessen self-assigned this Oct 24, 2024
@nickvergessen nickvergessen requested review from ArtificialOwl, artonge and yemkareems and removed request for a team October 24, 2024 20:06
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 25, 2024
@nickvergessen nickvergessen merged commit 631f6d4 into master Oct 25, 2024
174 of 177 checks passed
@nickvergessen nickvergessen deleted the bugfix/noid/allow-to-force-db-throttler branch October 25, 2024 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SystemKeeper SystemKeeper SystemKeeper approved these changes

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl was automatically assigned from nextcloud/server-backend

@artonge artonge Awaiting requested review from artonge artonge was automatically assigned from nextcloud/server-backend

@yemkareems yemkareems Awaiting requested review from yemkareems yemkareems was automatically assigned from nextcloud/server-backend

Assignees

@nickvergessen nickvergessen

Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement
Projects
None yet
Milestone
Nextcloud 31
Development

Successfully merging this pull request may close these issues.
3 participants
@nickvergessen @ChristophWurst @SystemKeeper