improved ldap errors checking

Etienne Pallier
1 parent f37b9499
Showing 1 changed file with 5 additions and 3 deletions Show diff stats
src/Model/Table/LdapConnectionsTable.php
@@ -357,14 +357,16 @@ class LdapConnectionsTable extends AppTable
                     $binddn .= ','.$this->baseDn;
  
                     // Connection
-                    $ldapConnection = ldap_connect($this->host, $this->port);
+                    $ldapConnection = ldap_connect($this->host, $this->port) or die("Could not connect to $this->host (port $this->port)");
                     if ($ldapConnection) {
                         ldap_set_option($ldapConnection, LDAP_OPT_PROTOCOL_VERSION, 3);
                         // Binding
                         //if (@ldap_bind($ldapConnection, $this->authenticationType . '=' . $login . ',' . $this->baseDn, $password)) {
-                        $ldapbind = ldap_bind($ldapConnection, $binddn, $ldappass);
+                        $ldapbind = ldap_bind($ldapConnection, $binddn, $ldappass) or die( "Could not bind to LDAP server.". ldap_error($ldapConnection) );
                         if ($ldapbind) {
-                            return $this->getUserAttributes($login, $LDAP_ANONYMOUS, $filter, $just_these)[0];
+                            //return $this->getUserAttributes($login, $LDAP_ANONYMOUS, $filter, $just_these)[0];
+                            $search = $this->getUserAttributes($login, $LDAP_ANONYMOUS, $filter, $just_these) or die("Could not search in LDAP server, response was: " . ldap_error($ldapConnection) );
+                            return $search[0];
                             /*
                              * } else {
                              * return false;