Opened 11 years ago

Last modified 14 months ago

#10700 closed defect

AccountModule._do_reset_password discards errors from _reset_password — at Version 1

Reported by: anonymous Owned by: Steffen Hoffmann
Priority: normal Component: AccountManagerPlugin
Severity: critical Keywords: AccountModule
Cc: Ryan J Ollos Trac Release:

Description (last modified by Steffen Hoffmann)

In web_ui.py, in the AccountModule class is a function _reset_password, which can return an error. _do_reset_password always ignores the error, and instead reports success to the user. As far as I can tell, these errors are always ignored.

This then makes it extremely difficult to diagnose other problems with password reset.

Change History (1)

comment:1 Changed 11 years ago by Steffen Hoffmann

Cc: Ryan J Ollos added; anonymous removed
Description: modified (diff)
Keywords: AccountModule added
Summary: Reset passport discards errorsAccountModule._do_reset_password discards errors from _reset_password

Good catch, issue verified here easily with your hint on how to reproduce (in #10701). Thanks for noticing.

As mentioned in my recent comment to the follow-up issue #10701 I'd appreciate disclose of a reporter contact. Anonymous reports just don't work that well for human interaction, you see?

Note: See TracTickets for help on using tickets.