Skip to content

Commit

Permalink
fix: properly set includeACL for cert services
Browse files Browse the repository at this point in the history
  • Loading branch information
rvazarkar committed Jan 16, 2024
1 parent 28361f3 commit 3808c5a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Producers/LdapProducer.cs
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ await OutputChannel.Writer.WriteAsync(new Domain
foreach (var searchResult in Context.LDAPUtils.QueryLDAP(ldapData.Filter.GetFilter(), SearchScope.Subtree,
ldapData.Props.Distinct().ToArray(), cancellationToken, domain.Name,
adsPath: Context.SearchBase,
includeAcl: (Context.ResolvedCollectionMethods & ResolvedCollectionMethod.ACL) != 0))
includeAcl: (Context.ResolvedCollectionMethods & ResolvedCollectionMethod.ACL) != 0 || (Context.ResolvedCollectionMethods & ResolvedCollectionMethod.CertServices) != 0))
{
var l = searchResult.DistinguishedName.ToLower();
if (l.Contains("cn=domainupdates,cn=system"))
Expand Down Expand Up @@ -114,7 +114,7 @@ public override async Task ProduceConfigNC()
foreach (var searchResult in Context.LDAPUtils.QueryLDAP(configNcData.Filter.GetFilter(), SearchScope.Subtree,
configNcData.Props.Distinct().ToArray(), cancellationToken, domain.Name,
adsPath: configAdsPath,
includeAcl: (Context.ResolvedCollectionMethods & ResolvedCollectionMethod.ACL) != 0))
includeAcl: (Context.ResolvedCollectionMethods & ResolvedCollectionMethod.ACL) != 0 || (Context.ResolvedCollectionMethods & ResolvedCollectionMethod.CertServices) != 0))
{
await Channel.Writer.WriteAsync(searchResult, cancellationToken);
Context.Logger.LogTrace("Producer wrote {DistinguishedName} to channel", searchResult.DistinguishedName);
Expand Down

0 comments on commit 3808c5a

Please sign in to comment.