Langsam AD-Gruppenmitgliedschaft Lookup

Question

Ich habe einige Codes zu überprüfen, ob ein Domain-Benutzer ein Mitglied der Maschinen Gruppe Administratoren:

public static bool ActiveDirectoryGroupMembershipOk(string userid, string groupName) 
{ 
    using (PrincipalContext ctx = new PrincipalContext(ContextType.Machine, "my_pc_name")) 
    { 
     using (GroupPrincipal grp = GroupPrincipal.FindByIdentity(ctx, IdentityType.SamAccountName, "administrators")) 
     { 
      if (grp != null) 
      { 
       foreach (Principal p in grp.GetMembers(false)) 
       { 
        if (p is UserPrincipal && p.SamAccountName.Equals(userid, StringComparison.InvariantCultureIgnoreCase)) 
        { 
         return true; 
        } 
       } 
      } 
     } 
    } 
    return false; 
} 

Es funktioniert, aber die folgende Code-Zeile dauert einige Sekunden dauern:

using (GroupPrincipal grp = GroupPrincipal.FindByIdentity(ctx, IdentityType.SamAccountName, "administrators")) 

Gibt es einen schnelleren Weg, die Mitgliedschaft zu suchen?

Ich weiß nicht, ob es wichtig ist oder nicht, aber die Benutzer-ID ist ein Domänenbenutzer und die Windows-Gruppe ist auf dem lokalen PC.

Answer 1

Ich fand, dass es schneller scheint, den Benutzer nicht in einer Gruppe nachzuschlagen, sondern stattdessen die Rollenmitgliedschaft des Benutzers zu überprüfen.

Hier ist der Code, der in meiner Frage schneller als der Code ausführt:

public static bool ActiveDirectoryGroupMembershipOk(string userid, string groupName) 
{ 
    bool membershipOk = false; 
    using (var pc = new PrincipalContext(ContextType.Machine, "my_pc_name")) 
    { 
     using (var p = Principal.FindByIdentity(pc, IdentityType.SamAccountName, userid)) 
     { 
      // if user account exists, check the group membership 
      if(p != null) 
      { 
       System.Security.Principal.WindowsIdentity wi = new System.Security.Principal.WindowsIdentity(userid); 
       System.Security.Principal.WindowsPrincipal wp = new System.Security.Principal.WindowsPrincipal(wi); 
       membershipOk = wp.IsInRole(groupName); 
      } 
     } 
    } 
    return membershipOk; 
} 

Answer 2

ich einen besseren Weg gefunden, dies noch zu tun (eine AD-Domäne angenommen wird), von Mr. Blonder Antwort unter Verwendung eines Teils:

public static bool ActiveDirectoryGroupMembershipOk(String userid, String groupname) 
{ 
    PrincipalContext ctx = new PrincipalContext(ContextType.Domain); 
    UserPrincipal user = UserPrincipal.FindByIdentity(ctx, IdentityType.SamAccountName, userid); 
    GroupPrincipal group = GroupPrincipal.FindByIdentity(ctx, IdentityType.SamAccountName, groupname); 

    return user.IsMemberOf(group); 
}