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 5 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

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.