View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0013690 | mantisbt | ldap | public | 2011-12-19 08:51 | 2014-09-23 18:05 |
Reporter | VeMag | Assigned To | dregad | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 1.2.8 | ||||
Target Version | 1.2.9 | Fixed in Version | 1.2.9 | ||
Summary | 0013690: When using LDAP, administrator can't reset "failed_login_count" any more | ||||
Description | using LDAP and $g_max_failed_login_count=ON; Before, when user failed to login 3 times, he had to call administrator: Now, he has to use SQL on the database... That's since fix of the issue 0012998, had been better just change button text, not hide it. | ||||
Tags | No tags attached. | ||||
As a workaround, you could override the auth_can_change_password() custom function so that it returns true for LDAP. I agree that we should offer the admin an option to "Unlock the account", even in case Mantis can't reset the password. |
|
Can you test this? |
|
Great job ! it work's perfectly, good idea to hide/show the button, so the administor know the account was lock in mantis not in LDAP. Thanks a lot ! French traductions : |
|
Merci pour la traduction :-) I was also wondering if it would not be useful to show locked accounts on the user list page. |
|
dregad, it appears as if you're calling user_reset_password($p_user_id) twice in manage_user_reset.php? |
|
Thanks for feedback, I'll have a look |
|
I removed the extra call, thanks for catching this oversight. |
|
Marking as 'acknowledged' not resolved/closed to track that change gets ported to master-2.0.x branch |
|
MantisBT: master-1.2.x cc619820 2011-12-19 04:29 Details Diff |
Add Account Unlock button to manage_user_edit_page.php The button is displayed when it is not possible to reset the user's password as defined by custom function auth_can_change_password(), $g_max_failed_login_count > 0 and the user's failed login count is higher than this threshold. Prior to this, in the case where it is not possible to reset a user's password and $g_max_failed_login_count > 0, the administrator's only way to reset the failed login count was direct SQL update. This was introduced by commit 3eaa71f7711be41610fedb2d53506483261d8351. Fixes 0013690 |
Affected Issues 0013690 |
|
mod - lang/strings_english.txt | Diff File | ||
mod - manage_user_edit_page.php | Diff File | ||
mod - manage_user_reset.php | Diff File | ||
MantisBT: master 42e29db3 2011-12-31 09:34 Details Diff |
Add Account Unlock button to manage_user_edit_page.php The button is displayed when it is not possible to reset the user's password as defined by custom function auth_can_change_password(), $g_max_failed_login_count > 0 and the user's failed login count is higher than this threshold. Prior to this, in the case where it is not possible to reset a user's password and $g_max_failed_login_count > 0, the administrator's only way to reset the failed login count was direct SQL update. This was introduced by commit aa48e0c7547cd0edc4df95ff28f67d6f7b3d36a5. Fixes 0013690 |
Affected Issues 0013690 |
|
mod - lang/strings_english.txt | Diff File | ||
mod - manage_user_edit_page.php | Diff File | ||
mod - manage_user_reset.php | Diff File |