Merge pull request #16 from tsmgeek/master

Fix issue where loginattrib was fixed
hydrian · Nov 6, 2015 · 20457fc · 20457fc
2 parents 71b80bb + 2a1bdd0
commit 20457fc
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/plugins/auth_ldap/init.php b/plugins/auth_ldap/init.php
@@ -365,10 +365,10 @@ function authenticate($login, $password) {
             $loginAttempt = @ldap_bind($ldapConn, $userDN, $password);
             if ($loginAttempt === TRUE) {
                 $this->_log('User: ' . (string) $login . ' authentication successful');
-                if (strlen(userAttributes) > 0) {
+                if (strlen($this->_ldapLoginAttrib) > 0) {
                     if ($this->_debugMode)
                         $this->_log('Looking up TT-RSS username attribute in ' . $this->_ldapLoginAttrib);
-                    $ttrssUsername = $userAttributes['sAMAccountName'][0];
+                    $ttrssUsername = $userAttributes[$this->_ldapLoginAttrib][0];
                     ;
                     @ldap_close($ldapConn);
                     if (!is_string($ttrssUsername)) {