add login failure events to django.security logger
|Reported by:||ptone||Owned by:||nobody|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
#18616 added a signal for failed login attempts - a primary use-case for this was monitoring.
With #19866 we have the introduction of a django.security logger, and it would make more sense to log these events there.
I'd be +0 on also deprecating the signal, they are brand new, and probably don't have much traction. There may be non-monitoring uses of said signal though that I'm not thinking of.
Change History (19)
Changed 21 months ago by boylea
comment:2 Changed 21 months ago by ptone
- Has patch set
- Needs documentation set
- Patch needs improvement set
Changed 20 months ago by boylea
comment:13 Changed 4 months ago by berkerpeksag
- Needs documentation unset
- Patch needs improvement unset