mirror/librenms-librenms
1
0
Fork 0
mirror of https://github.com/librenms/librenms.git synced 2024-10-07 16:52:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Remove Success output from homepage when using AD Auth.

Browse Source
This commit is contained in:
Tony Murray
2016-01-25 13:10:51 -06:00
parent 858dbfbe79
commit b9448a61a8
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
html/includes/authentication/active_directory.inc.php
View File

@ -18,9 +18,9 @@ ldap_set_option($ds, LDAP_OPT_PROTOCOL_VERSION, 3);
function authenticate($username, $password) { function authenticate($username, $password) {
global $config, $ds; global $config, $ds;
if ($username && $ds) { if ($ds) {
// bind with sAMAccountName instead of full LDAP DN // 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 // group membership in one of the configured groups is required
if (isset($config['auth_ad_require_groupmembership']) && if (isset($config['auth_ad_require_groupmembership']) &&
$config['auth_ad_require_groupmembership'] > 0) { $config['auth_ad_require_groupmembership'] > 0) {