Modify ↓

Opened 13 years ago

Closed 12 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 13 years ago.
Patch for adauth.auth

Download all attachments as: .zip

Change History (3)

Changed 13 years ago by E Keith Thomson

Attachment: testdiff.patch​ added

Patch for adauth.auth

comment:1 Changed 13 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 12 years ago by branson

Resolution: → fixed
Status: new → closed

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.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.