X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=ldapauth%2Fldapauth.php;h=60273c3b824d29398e10471f4e403d923368ee2f;hb=e6b170da7de53df162fafdabbc1b00a31e709a3b;hp=62f4b003b09fbcdb0970fd729de77910a31f0125;hpb=31456b2f43daa3516300c69c4ad54d5468dece0c;p=friendica-addons.git

diff --git a/ldapauth/ldapauth.php b/ldapauth/ldapauth.php
index 62f4b003..60273c3b 100755
--- a/ldapauth/ldapauth.php
+++ b/ldapauth/ldapauth.php
@@ -47,7 +47,7 @@
  * //   attribute to get email - optional - default : 'mail'
  * $a->config['ldapauth']['ldap_autocreateaccount_emailattribute'] = 'mail';
  * //   attribute to get nickname - optional - default : 'givenName'
- * $a->config['ldapauth']['ldap_autocreateaccount_nameattribute'] = 'givenName';
+ * $a->config['ldapauth']['ldap_autocreateaccount_nameattribute'] = 'cn';
  *
  * ...etc.
  */
@@ -66,11 +66,14 @@ function ldapauth_uninstall() {
 
 
 function ldapauth_hook_authenticate($a,&$b) {
-	if(ldapauth_authenticate($b['username'],$b['password']) && is_existing_account($b['username'])) {
-		$b['user_record'] = $results[0];
-		$b['authenticated'] = 1;
-	}
-	return;
+    if(ldapauth_authenticate($b['username'],$b['password'])) {
+    	$results = get_existing_account($b['username']);
+    	if(! empty($results)){
+            $b['user_record'] = $results[0];
+            $b['authenticated'] = 1;
+    	}
+    }
+    return;
 }
 
 function ldapauth_authenticate($username,$password) {
@@ -81,34 +84,35 @@ function ldapauth_authenticate($username,$password) {
     $ldap_searchdn = get_config('ldapauth','ldap_searchdn');
     $ldap_userattr = get_config('ldapauth','ldap_userattr');
     $ldap_group    = get_config('ldapauth','ldap_group');
-	$ldap_autocreateaccount = get_config('ldapauth','ldap_autocreateaccount');
-	$ldap_autocreateaccount_emailattribute = get_config('ldapauth','ldap_autocreateaccount_emailattribute');
-	$ldap_autocreateaccount_nameattribute = get_config('ldapauth','ldap_autocreateaccount_nameattribute');
-
-	if(! strlen($ldap_autocreateaccount_emailattribute))
-		$ldap_autocreateaccount_emailattribute = "mail";
-	if(! strlen($ldap_autocreateaccount_nameattribute))
-		$ldap_autocreateaccount_nameattribute = "givenName";
+    $ldap_autocreateaccount = get_config('ldapauth','ldap_autocreateaccount');
+    $ldap_autocreateaccount_emailattribute = get_config('ldapauth','ldap_autocreateaccount_emailattribute');
+    $ldap_autocreateaccount_nameattribute = get_config('ldapauth','ldap_autocreateaccount_nameattribute');
 	
     if(! ((strlen($password))
             && (function_exists('ldap_connect'))
-            && (strlen($ldap_server))))
+            && (strlen($ldap_server)))) {
+            logger("ldapauth: not configured or missing php-ldap module");
             return false;
+    }
 
     $connect = @ldap_connect($ldap_server);
 
-    if(! $connect)
+    if($connect === false) {
+        logger("ldapauth: could not connect to $ldap_server");
         return false;
+    }
 
     @ldap_set_option($connect, LDAP_OPT_PROTOCOL_VERSION,3);
     @ldap_set_option($connect, LDAP_OPT_REFERRALS, 0);
     if((@ldap_bind($connect,$ldap_binddn,$ldap_bindpw)) === false) {
+        logger("ldapauth: could not bind $ldap_server as $ldap_binddn");
         return false;
     }
 
     $res = @ldap_search($connect,$ldap_searchdn, $ldap_userattr . '=' . $username);
 
     if(! $res) {
+        logger("ldapauth: $ldap_userattr=$username,$ldap_searchdn not found");
         return false;
     }
 
@@ -122,14 +126,22 @@ function ldapauth_authenticate($username,$password) {
 
     if(! @ldap_bind($connect,$dn,$password))
         return false;
-		
-	$emailarray = @ldap_get_values($connect, $id, $ldap_autocreateaccount_emailattribute);
-	$namearray = @ldap_get_values($connect, $id, $ldap_autocreateaccount_nameattribute);
+    
+    $emailarray = [];
+    $namearray = [];
+    if($ldap_autocreateaccount == "true"){
+        if(! strlen($ldap_autocreateaccount_emailattribute))
+            $ldap_autocreateaccount_emailattribute = "mail";
+        if(! strlen($ldap_autocreateaccount_nameattribute))
+            $ldap_autocreateaccount_nameattribute = "givenName";
+    	$emailarray = @ldap_get_values($connect, $id, $ldap_autocreateaccount_emailattribute);
+    	$namearray = @ldap_get_values($connect, $id, $ldap_autocreateaccount_nameattribute);
+    }
 
     if(! strlen($ldap_group)){
-		ldap_autocreateaccount($ldap_autocreateaccount,$username,$password,$emailarray[0],$namearray[0]);
+    	ldap_autocreateaccount($ldap_autocreateaccount,$username,$password,$emailarray[0],$namearray[0]);
         return true;
-	}
+    }
 
     $r = @ldap_compare($connect,$ldap_group,'member',$dn);
     if ($r === -1) {
@@ -155,30 +167,29 @@ function ldapauth_authenticate($username,$password) {
         return false;
     }
 
-	ldap_autocreateaccount($ldap_autocreateaccount,$username,$password,$emailarray[0],$namearray[0]);
+    ldap_autocreateaccount($ldap_autocreateaccount,$username,$password,$emailarray[0],$namearray[0]);
     return true;
 }
 
 function ldap_autocreateaccount($ldap_autocreateaccount,$username,$password,$email,$name) {
-	if($ldap_autocreateaccount == "true" && !is_existing_account($username)){
-		if (strlen($email) > 0 && strlen($name) > 0){
-			$arr = array('username'=>$name,'nickname'=>$username,'email'=>$email,'password'=>$password,'verified'=>1);
-			$result = create_user($arr);
-			if ($result['success']){
-				logger("ldapauth: account " . $username . " created");
-			}else{
-				logger("ldapauth: account " . $username . " was not created ! : " . implode($result));
-			}
-		}else{
-			logger("ldapauth: unable to create account, no email or nickname found");
-		}
-	}
+    if($ldap_autocreateaccount == "true"){
+        $results = get_existing_account($username);
+        if(empty($results)){
+            if (strlen($email) > 0 && strlen($name) > 0){
+                $arr = array('username'=>$name,'nickname'=>$username,'email'=>$email,'password'=>$password,'verified'=>1);
+                $result = create_user($arr);
+                if ($result['success']){
+                    logger("ldapauth: account " . $username . " created");
+                }else{
+                    logger("ldapauth: account " . $username . " was not created ! : " . implode($result));
+                }
+            }else{
+                logger("ldapauth: unable to create account, no email or nickname found");
+            }
+        }
+    }
 }
 
-function is_existing_account($username){
-	$results = q("SELECT * FROM `user` WHERE `nickname` = '%s' AND `blocked` = 0 AND `verified` = 1 LIMIT 1",$username);
-	if(count($results)) {
-		return true;
-	}
-	return false;
+function get_existing_account($username){
+    return q("SELECT * FROM `user` WHERE `nickname` = '%s' AND `blocked` = 0 AND `verified` = 1 LIMIT 1",$username);
 }