fix: Alter ConverDomainSidToDomainNameFromLdap - BED-6095 (#287)

* fix: Alter ConverDomainSidToDomainNameFromLdap to return proper domain in cross forest scenario

* test: Add WindowsOnlyFact to Test

Author: mykeelium

src/CommonLib/LdapUtils.cs

@@ -412,14 +412,14 @@ public IAsyncEnumerable<LdapResult<IDirectoryObject>> PagedQuery(LdapQueryParame
 
             result = await Query(new LdapQueryParameters {
                 DomainName = domain.Name,
-                Attributes = new[] { LDAPProperties.DistinguishedName },
+                Attributes = new[] { LDAPProperties.DistinguishedName, LDAPProperties.Name },
                 GlobalCatalog = true,
                 LDAPFilter = new LdapFilter().AddFilter("(objectclass=trusteddomain)", true)
                     .AddFilter($"(securityidentifier={Helpers.ConvertSidToHexSid(domainSid)})", true).GetFilter()
             }).DefaultIfEmpty(LdapResult<IDirectoryObject>.Fail()).FirstOrDefaultAsync();
 
-            if (result.IsSuccess && result.Value.TryGetDistinguishedName(out distinguishedName)) {
-                return (true, Helpers.DistinguishedNameToDomain(distinguishedName));
+            if (result.IsSuccess && result.Value.TryGetProperty(LDAPProperties.Name, out var domainName)) {
+                return (true, domainName.ToUpper());
             }
 
             result = await Query(new LdapQueryParameters {

test/unit/WebClientServiceProcessorTest.cs

@@ -26,7 +26,7 @@ public WebClientServiceProcessorTest(ITestOutputHelper testOutputHelper) {
         public void Dispose() {
         }
 
-        [Fact]
+        [WindowsOnlyFact]
         public async Task WebClientServiceProcessorTest_TestPathExists()
         {
             var processor = new WebClientServiceProcessor();