Modify

Opened 6 years ago

Closed 5 years ago

#9112 closed defect (fixed)

AccountManager expects boolean, gets ldap object.

Reported by: E Keith Thomson Owned by: John Hampton
Priority: normal Component: DirectoryAuthPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.12

Description

I've encountered an issue with the current version of Account manager. We traced it down to ActiveDirectoryAuthPlugin providing a ldap object to indicate success of password verification, and AccountManager failing because it was expecting either True or False. Attached is a patch that allows this to work again.

Attachments (1)

testdiff.patch (398 bytes) - added by E Keith Thomson 6 years ago.
Patch for adauth.auth

Download all attachments as: .zip

Change History (3)

Changed 6 years ago by E Keith Thomson

Attachment: testdiff.patch added

Patch for adauth.auth

comment:1 Changed 6 years ago by olaf.meeuwissen@…

Same thing here. Without this patch I'd see in the logs that the Password Verified in the plugin but got an Invalid username or password from AccountManager.LoginModule.process_request() anyway.

comment:2 Changed 5 years ago by branson

Resolution: fixed
Status: newclosed

integrated into 0.3

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain John Hampton.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.