Skip to content

feat: do not report TooManyLoginAttemptsError to Sentry#1312

Merged
suricactus merged 1 commit intomasterfrom
QF-6525-ignore-too-many-login
Aug 26, 2025
Merged

feat: do not report TooManyLoginAttemptsError to Sentry#1312
suricactus merged 1 commit intomasterfrom
QF-6525-ignore-too-many-login

Conversation

@suricactus
Copy link
Collaborator

We cannot do much about it. Probably we can just monitor if there are way too many of them per hour or so in the monitoring.

We cannot do much about it. Probably we can just monitor if there are way too many of them per hour or so in the monitoring.
@duke-nyuki
Copy link
Collaborator

@gounux
Copy link
Member

gounux commented Aug 25, 2025

What if there is a surprisingly high number of login requests ? Will we still be available to see it go by ?

@suricactus
Copy link
Collaborator Author

We will be able to see it from Loki if we configure it there. Nothing by default.

@suricactus suricactus requested a review from gounux August 25, 2025 22:23
@suricactus suricactus merged commit ff48533 into master Aug 26, 2025
21 checks passed
@suricactus suricactus deleted the QF-6525-ignore-too-many-login branch August 26, 2025 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants