bugphpGroupWare - Bugs: bug #4462, Change Password fails for LDAP...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #4462: Change Password fails for LDAP auth when using SQL accounts

Submitter:  Invalid User ID <#21226>
Submitted:  Thu 24 Jul 2003 06:40:38 PM UTC
   
 
Category:  API - phpGroupware API Item Group:  0.9.14.004/5/6 release
Severity:  3 - Normal Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  Caeies Open/Closed:  Open
Component Version:  None Operating System:  None
Reproducibility:  Every Time Planned Release:  None
Fixed Release: 

Wed 26 Oct 2005 08:50:05 AM UTC, comment #3: 

Benoit you might want to look at this at some stage

Dave Hall <skwashd>
Group administrator
Thu 23 Oct 2003 09:44:33 PM UTC, comment #2: 

Is this still a problem?

Hsing-Foo Wang <pooh_>
Wed 22 Oct 2003 04:43:04 PM UTC, comment #1: 

Note:  The attached patch needs further work.  It appears to break the case when you change another user's password from the Admin -> User Accounts menu.

Invalid User ID <#21226>
Thu 24 Jul 2003 06:40:38 PM UTC, original submission:  

We using LDAP authentication, but storing our accounts in MySQL.  This scenario works fine so far, except for one thing.  When a user changes their password, it appears to work, but really fails.
                                                                               
auth_ldap is trying to find the user entry in the director by filtering on uidnumber=$_account_id.  This is fine if accounts are in LDAP.  However, if the accounts are not in the directory, they will not match.  In this case, we should instead filter by looking for a uid that matches the PHPGW user ID.
                                                                               
I've modified class.auth_ldap.inc.php to check and see if the account repository is ldap.  If it is, it functions as before.  If it's not, then it filters on "uid=".$GLOBALS['phpgw_info']['user']['userid'].

Another option would be to just always filter via uid, even for accounts == ldap.
                                                                               
I've attached the modified class.auth_ldap.inc.php as well as the output of cvs diff class.auth_ldap.inc.php.
                                                                               
It has been tested with both auth = LDAP and accounts = SQL and auth = LDAP and accounts = LDAP
                                                                               
Thanks,
                                                                               
Doug

Invalid User ID <#21226>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

Follow 2 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2005-10-26 skwashd Assigned tofipsfuchs Caeies
2004-01-15 powerstat Assigned toNone fipsfuchs

Back to the top

Powered by Savane 3.13-b921.
Corresponding source code