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 .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from pacopablo. Next status will be '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.