Solved

How do I get All Active Directory Members of Nested Groups - PHP

Posted on 2008-10-17
2
2,626 Views
Last Modified: 2013-12-24
In Active Directory I would like to find ALL physical users that are in the Group, for example:
OU=Distribution Lists,OU=My_OU,DC=my-connection,DC=ch

Obtaining one level is rather straightforward - I get physical users and more Groups if they are present. I am using 'member' property for that.
Problem starts with going deeper, in scenario when Group contains Groups (which can then contain other Groups or finally physical users I am looking for). I wrote a recursive function for that, however I am getting only physical users that are just  one level down from Group I started a search from. While debugging I see that Groups are being processed by recursive function but are not passed to function's RETURN variable; only one level users are passed to this variable. What am I doing wrong ?
/*
$members  Members(s) of a group
$ad  resource $link_identifier
*/
 
function check_ad_data($members, $ad) 
{
  // get rid of array's 'count' key => value so not to include it for iteration
  unset($members['count']);
  
  foreach($members as $member) {
    
    // get entries of member from Active Directory
    $attributes = array("objectGUID", "mail", "name", "objectClass", "member");
    $filter = "(mail=*)";
    $result = ldap_search($ad, $member, $filter, $attributes); 
    $entries = ldap_get_entries($ad, $result);
    
    
    // if member is the Person
    if($entries[0]['objectclass'][1] == 'person') {
      // append a person's name to array
      $users[] = $entries[0]['name'][0];
    }
    
    // if member is the Group
    if($entries[0]['objectclass'][1] == 'group') {
      // obtain all members of this group
      $membersOfGroup = $entries[0]['member'];    
      // call a recursive function
      check_ad_data($membersOfGroup, $ad);
    }
    
  }
 
  // return All the users
  return $users;
 
}

Open in new window

0
Comment
Question by:greeni
  • 2
2 Comments
 
LVL 70

Accepted Solution

by:
Chris Dent earned 500 total points
ID: 22739135

Don't you have to set the return value to a variable?

e.g.

$users = check_ad_data ...

And if that is the case, shouldn't you also be doing that here:

    if($entries[0]['objectclass'][1] == 'group') {
      ...
      $users = check_ad_data($membersOfGroup, $ad);
    }

Because if $users only exists for the lifetime of the function (rather than globally) it will only update within the function. Maybe...

Chris
0
 
LVL 70

Expert Comment

by:Chris Dent
ID: 22739178

It's worth mentioning that the reason I'm not entirely sure about the above is that for anything like this I would use a global variable / array to store the results of the function. But then, I'm much more into the AD / administration side than the coding side :)

Chris
0

Featured Post

Live: Real-Time Solutions, Start Here

Receive instant 1:1 support from technology experts, using our real-time conversation and whiteboard interface. Your first 5 minutes are always free.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Since pre-biblical times, humans have sought ways to keep secrets, and share the secrets selectively.  This article explores the ways PHP can be used to hide and encrypt information.
This article shows how to deploy dynamic backgrounds to computers depending on the aspect ratio of display
Video by: Steve
Using examples as well as descriptions, step through each of the common simple join types, explaining differences in syntax, differences in expected outputs and showing how the queries run along with the actual outputs based upon a simple set of dem…
This video shows how to use Hyena, from SystemTools Software, to bulk import 100 user accounts from an external text file. View in 1080p for best video quality.

776 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question