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

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

greeniAsked:
Who is Participating?
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

Chris DentPowerShell DeveloperCommented:

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

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
Chris DentPowerShell DeveloperCommented:

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
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
Databases

From novice to tech pro — start learning today.

Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.