Modify

Opened 3 years ago

Closed 2 years ago

#9112 closed defect (fixed)

AccountManager expects boolean, gets ldap object.

Reported by: ekthomson Owned by: pacopablo
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 ekthomson 3 years ago.
Patch for adauth.auth

Download all attachments as: .zip

Change History (3)

Changed 3 years ago by ekthomson

Patch for adauth.auth

comment:1 Changed 3 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 2 years ago by sandinak

  • Resolution set to fixed
  • Status changed from new to closed

integrated into 0.3

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.