Opened 8 years ago

Last modified 8 years ago

#27902 closed Bug

user_can_authenticate prevents confirm_login_allowed from checking inactive user — at Initial Version

Reported by: Brandon Chinn Owned by: nobody
Component: contrib.auth Version: 1.10
Severity: Normal Keywords: authentication, inactive
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

With the addition of ModelBackend.user_can_authenticate, ModelBackend.authenticate now returns None if a user is inactive. Because of that, AuthenticationForm.clean will set self.user_cache to be None and will error with the invalid_login message instead of the inactive message.

Not sure how we want this to be solved, but I'll try to put up a patch resolving this issue and see if that's the direction we want to take.

Change History (0)

Note: See TracTickets for help on using tickets.
Back to Top