composer/iTop
6
0
Fork 0
mirror of https://github.com/Combodo/iTop.git synced 2026-05-19 15:22:17 +02:00
Code Issues Packages Projects Releases Wiki Activity

N°3472 - Refactor LDAP debug level handling to ensure proper logging during connection attempts (#892)

Browse Source
This commit is contained in:
Lenaick
2026-04-27 14:26:38 +02:00
committed by GitHub
parent d271dda54f
commit befe294986
1 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
datamodels/2.x/authent-ldap/datamodel.authent-ldap.xml
View File

@@ -97,18 +97,19 @@
$bDebug = isset($aServerParams['debug']) ? $aServerParams['debug'] : false; $bDebug = isset($aServerParams['debug']) ? $aServerParams['debug'] : false;
} }
if (array_key_exists(LDAP_OPT_DEBUG_LEVEL, $aOptions))
{
// Set debug level before trying to connect, so that debug info appear in the PHP error log if ldap_connect goes wrong
$bRet = ldap_set_option(null, LDAP_OPT_DEBUG_LEVEL, $aOptions[LDAP_OPT_DEBUG_LEVEL]);
$this->LogInfo($bDebug, "ldap_set_option('LDAP_OPT_DEBUG_LEVEL', '{$aOptions[LDAP_OPT_DEBUG_LEVEL]}') returned ".($bRet ? 'true' : 'false'));
}
$hDS = @ldap_connect($sURI); $hDS = @ldap_connect($sURI);
if ($hDS === false) if ($hDS === false)
{ {
$this->LogIssue($bDebug, "ldap_authentication: can not connect to the LDAP server '$sURI'. Check the configuration file config-itop.php."); $this->LogIssue($bDebug, "ldap_authentication: can not connect to the LDAP server '$sURI'. Check the configuration file config-itop.php.");
return false; return false;
} }
if (array_key_exists(LDAP_OPT_DEBUG_LEVEL, $aOptions))
{
// Set debug level before trying to connect, so that debug info appear in the PHP error log if ldap_connect goes wrong
$bRet = ldap_set_option($hDS, LDAP_OPT_DEBUG_LEVEL, $aOptions[LDAP_OPT_DEBUG_LEVEL]);
$this->LogInfo($bDebug, "ldap_set_option('LDAP_OPT_DEBUG_LEVEL', '{$aOptions[LDAP_OPT_DEBUG_LEVEL]}') returned ".($bRet ? 'true' : 'false'));
}
foreach($aOptions as $name => $value) foreach($aOptions as $name => $value)
{ {
$bRet = ldap_set_option($hDS, $name, $value); $bRet = ldap_set_option($hDS, $name, $value);