Skip to content

Commit

Permalink
Merge pull request netwrix#17 from cnotin/patch-4
Browse files Browse the repository at this point in the history
"DnsAdmins" group is under the "CN=Users" container by default (not at domain root)
  • Loading branch information
vletoux authored Oct 12, 2019
2 parents faa2cb0 + 757b641 commit 19adb81
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Healthcheck/HealthcheckAnalyzer.cs
Original file line number Diff line number Diff line change
Expand Up @@ -1159,7 +1159,7 @@ private void GeneratePrivilegedGroupData(ADDomainInfo domainInfo, ADWebService a
dnsAdminFound = true;
};
// we do a one level search just case the group is in the default position
adws.Enumerate(domainInfo.DefaultNamingContext, "(&(objectClass=group)(description=DNS Administrators Group))", properties, callback, "OneLevel");
adws.Enumerate("CN=Users," + domainInfo.DefaultNamingContext, "(&(objectClass=group)(description=DNS Administrators Group))", properties, callback, "OneLevel");
if (!dnsAdminFound)
{
// then full tree. This is an optimization for LDAP request
Expand Down

0 comments on commit 19adb81

Please sign in to comment.