Question

I currently have some code that pulls down a list of users in a group and then iterates through that group to determine if a given account exists, but it seems like there ought to be a more concise (and perhaps faster) way to accomplish this.

This code (VB.NET) attempts to use the member property of the group object, but it is returning false even when the user is a member of that group. Can anyone see what I am doing wrong here?

Dim group As DirectoryEntry =  GetNetworkObject(GroupDomanName, NetworkObjectType.NetworkGroup, GroupName) Dim user As DirectoryEntry =GetNetworkObject(UserDomainName, NetworkObjectType.NetworkUser, Login)  Return group.Properties('member').Contains(user.Path) 

FYI: The GetNetworkObject calls just return a directoryEntry object, I have confirmed that the correct object is being returned for both the group and user object.

Answer 1

If you are on .NET 3.5 stack, System.DirectoryServices.AccountManagement.dll assembly has a nice API on top of AD. The following method can be implemented to solve your issue:

static bool IsUserMemberOf(string userName, string groupName) {     using (var ctx = new PrincipalContext(ContextType.Domain))     using (var groupPrincipal = GroupPrincipal.FindByIdentity(ctx, groupName))     using (var userPrincipal = UserPrincipal.FindByIdentity(ctx, userName))     {         return userPrincipal.IsMemberOf(groupPrincipal);     } }  // Usage: bool result = IsUserMemberOf('CONTOSO\\john.doe', 'CONTOSO\\Administrators'); 

I don’t know how this method performs but it is a clean solution.