lainwir3d/dolibarr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

FIX Option start tls of ldap was not correctly supported.

Browse Source
This commit is contained in:
Laurent Destailleur 2015-12-13 15:51:52 +01:00
parent ef33691818
commit 7db6ac592f
1 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
htdocs/core/class/ldap.class.php
View File

@ -149,7 +149,7 @@ class Ldap
*/
function connect_bind()
{
global $langs;
global $langs, $conf;
$connected=0;
$this->bind=0;
@ -186,7 +186,18 @@ class Ldap
if (is_resource($this->connection))
{
// Execute the ldap_set_option here (after connect and before bind)
// Begin TLS if requested by the configuration
if (! empty($conf->global->LDAP_SERVER_USE_TLS))
{
if (! ldap_start_tls($this->connection))
{
dol_syslog(get_class($this)."::connect_bind failed to start tls", LOG_WARNING);
$connected = 0;
$this->close();
}
}
// Execute the ldap_set_option here (after connect and before bind)
$this->setVersion();
ldap_set_option($this->connection, LDAP_OPT_SIZELIMIT, 0); // no limit here. should return true.