lunax/snipe_it
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity Actions

Merge pull request #16337 from joakimbergros/develop

Browse source 
Fixed #16173: `useraccountcontrol` was not included in the ldap query attributes
This commit is contained in:
snipe 2025-02-25 13:49:38 +00:00 • committed by GitHub
commit 7b4f4b6b7f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/Console/Commands/LdapSync.php
View file

@ -125,6 +125,10 @@ class LdapSync extends Command
*/ */
$attributes = array_values(array_filter($ldap_map)); $attributes = array_values(array_filter($ldap_map));
if (Setting::getSettings()->is_ad === 1 && is_null($ldap_map['active_flag'])) {
$attributes[] = 'useraccountcontrol';
}
$results = Ldap::findLdapUsers($search_base, -1, $filter, $attributes); $results = Ldap::findLdapUsers($search_base, -1, $filter, $attributes);
} catch (\Exception $e) { } catch (\Exception $e) {