From b9448a61a81ce3bf6d1444b1d26a0f33f788b648 Mon Sep 17 00:00:00 2001 From: Tony Murray Date: Mon, 25 Jan 2016 13:10:51 -0600 Subject: [PATCH] Remove Success output from homepage when using AD Auth. --- html/includes/authentication/active_directory.inc.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/html/includes/authentication/active_directory.inc.php b/html/includes/authentication/active_directory.inc.php index 9df49bacc..d936b44c8 100644 --- a/html/includes/authentication/active_directory.inc.php +++ b/html/includes/authentication/active_directory.inc.php @@ -18,9 +18,9 @@ ldap_set_option($ds, LDAP_OPT_PROTOCOL_VERSION, 3); function authenticate($username, $password) { global $config, $ds; - if ($username && $ds) { + if ($ds) { // bind with sAMAccountName instead of full LDAP DN - if (ldap_bind($ds, "{$username}@{$config['auth_ad_domain']}", $password)) { + if ($username && ldap_bind($ds, "{$username}@{$config['auth_ad_domain']}", $password)) { // group membership in one of the configured groups is required if (isset($config['auth_ad_require_groupmembership']) && $config['auth_ad_require_groupmembership'] > 0) {