From ef238e533214c8a3e9e6e4b4d9dfdd564bfd57e6 Mon Sep 17 00:00:00 2001
From: FreddleSpl0it <patschul@posteo.de>
Date: Wed, 28 Aug 2024 11:28:37 +0200
Subject: [PATCH] [LDAP] skip sync user if username_field in LDAP is empty

---
 data/conf/phpfpm/crons/ldap-sync.php | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/data/conf/phpfpm/crons/ldap-sync.php b/data/conf/phpfpm/crons/ldap-sync.php
index 1a53884c..93b22b94 100644
--- a/data/conf/phpfpm/crons/ldap-sync.php
+++ b/data/conf/phpfpm/crons/ldap-sync.php
@@ -146,6 +146,11 @@ foreach ($response as $user) {
     continue;
   }
 
+  if (empty($user[$iam_settings['username_field']][0])){
+    logMsg("warning", "Skipping user " . $user['displayname'][0] . " due to empty LDAP ". $iam_settings['username_field'] . "property.'");
+    continue;
+  }
+
   if (!$row && intval($iam_settings['import_users']) == 1){
     // mailbox user does not exist, create...
     logMsg("info", "Creating user " .  $user[$iam_settings['username_field']][0]);