0012747: Better log message for LDAP auth errors

ID	Project	Category	View Status	Date Submitted	Last Update
0012747	mantisbt	ldap	public	2011-02-05 19:14	2011-04-05 14:23

Reporter	jesusaplsoft	Assigned To	vboctor
Priority	normal	Severity	minor	Reproducibility	sometimes
Status	closed	Resolution	fixed
Product Version	1.2.4
Fixed in Version	1.2.5

Summary	0012747: Better log message for LDAP auth errors
Description	In file "core/ldap_api.php", lines 78-83 are: if ( !$t_br ) { log_event( LOG_LDAP, "bind to ldap server failed - authentication error?" ); trigger_error( ERROR_LDAP_AUTH_FAILED, ERROR ); } else { log_event( LOG_LDAP, "bind to ldap server successful" ); } and should be: if ( !$t_br ) { log_event( LOG_LDAP, "bind to ldap server failed: ".ldap_error($t_ds) ); trigger_error( ERROR_LDAP_AUTH_FAILED, ERROR ); } else { log_event( LOG_LDAP, "bind to ldap server successful" ); } The only change made is related to line 79 (2nd one). This way error cause is reported in logging file
Tags	patch

MantisBT: master 85439a65 2011-02-13 14:45 vboctor Details Diff	Fix 0012747: Better log message for LDAP auth errors.	Affected Issues 0012747
	mod - core/ldap_api.php	Diff File
MantisBT: master-1.2.x 75fef46e 2011-02-13 14:45 vboctor Details Diff	Fix 0012747: Better log message for LDAP auth errors.	Affected Issues 0012747
	mod - core/ldap_api.php	Diff File

jesusaplsoft 2011-02-05 19:33 reporter ~0028171	They're lines 73-78, actually and the bad one is number 74