From 779c90b1fd026f57e899419ae43b46912be82b6d Mon Sep 17 00:00:00 2001
From: Falk Stern <fstern@goodgamestudios.com>
Date: Fri, 30 Oct 2015 17:17:55 +0100
Subject: [PATCH] Checking for groups now

---
 .../authentication/active_directory.inc.php   | 24 ++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

diff --git a/html/includes/authentication/active_directory.inc.php b/html/includes/authentication/active_directory.inc.php
index dc4d11eb3..316017855 100644
--- a/html/includes/authentication/active_directory.inc.php
+++ b/html/includes/authentication/active_directory.inc.php
@@ -22,7 +22,29 @@ function authenticate($username, $password) {
     if ($username && $ds) {
         // bind with sAMAccountName instead of full LDAP DN
         if (ldap_bind($ds, "{$username}@{$config['auth_ad_domain']}", $password)) {
-            return 1;
+            // group membership in one of the configured groups is required
+            if (isset($config['auth_ad_require_groupmembership']) &&
+                $config['auth_ad_require_groupmembership'] > 0) {
+                $search = ldap_search($ds, $config['auth_ad_base_dn'],
+                                      "(samaccountname={$username})", array('memberOf'));
+                $entries = ldap_get_entries($ds, $search);
+
+                $user_authenticated = 0; 
+                
+                foreach ($entries[0]['memberof'] as $entry) {
+                    $group_cn = get_cn($entry);
+                    if (isset($config['auth_ad_groups'][$group_cn]['level'])) {
+                        // user is in one of the defined groups
+                        $user_authenticated = 1;
+                    } 
+                }
+
+                return $user_authenticated;
+                
+            } else {
+                // group membership is not required and user is valid
+                return 1;
+            };
         } else {
             return 0;
         }