patchphpGroupWare - Patches: patch #3333, ENH: LDAP Better error...

 
 

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

patch #3333: ENH: LDAP Better error messaging/handling

Submitter:  None
Submitted:  Wed 01 Sep 2004 04:58:01 PM UTC
 
Category:  None Priority:  5 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned to:  None
Originator Email:  -email is unavailable- Open/Closed:  Open

Fri 01 Oct 2004 07:07:46 PM UTC, comment #3: 

I agree. I submitted that patch a bit hastily. I have come up with a solution for the mean time: create a seperate branch in LDAP for groupware groups.

Greg Valcourt <gval>
Wed 29 Sep 2004 06:53:12 AM UTC, comment #2: 

Unfortunately, the SQL code is not compatible with this kind of thing.
We need to rewrite the SQL way of handling groups and accounts in SQL to be able to change them in account_ldap
This could be a problem for example, when trying to switch from LDAP to SQL like authentification.

Regards,

Caeies.

Caeies <Caeies>
Project Administrator
Wed 01 Sep 2004 06:01:05 PM UTC, comment #1: 

Target Branch: 0.9.16, HEAD
Purpose: More thorough error messages in ldapmodify, account/group id number conflict should not be a fatal error.
Format: diff
Affected file/s: setup/ldapmodify.php, phpgwapi/inc/class.accounts_ldap.inc.php

ldapmodify now prints dn used, and what the error returned from the ldap_mod_replace function was when adding a user already in ldap.

The login page was giving a fatal error when you tried to login and you had an uid number that was the same as a gid in ldap. uid's and gid's are not related and you will usually find many uid's that are the same number as a gid. I changed the get_type() function to return type as 'u' when both numbers exist. If this is a problem, perhaps consider having collissions generate warnings.

Anonymous
Wed 01 Sep 2004 04:58:01 PM UTC, original submission:  
Anonymous

 

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

Attached Files
file #7825:  patchfile.diff added by None (3KiB - text/x-patch)

 

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.

 

 

 

Follows 1 latest change.

Date Changed by Updated Field Previous Value => Replaced by
2004-09-01 None Attached File- Added patchfile.diff, #3629

Back to the top

Powered by Savane 3.9