Browse Source

Signed-off-by: Russ Whitehead <william.whitehead@lexisnexis.com>

Russ Whitehead 7 years ago
parent
commit
f3746a96a3
1 changed files with 2 additions and 4 deletions
  1. 2 4
      system/security/LdapSecurity/ldapsecurity.cpp

+ 2 - 4
system/security/LdapSecurity/ldapsecurity.cpp

@@ -681,7 +681,7 @@ bool CLdapSecManager::authenticate(ISecUser* user)
         else
         {
             user->setAuthenticateStatus(AS_AUTHENTICATED);
-            if(isCaching)
+            if(isCaching && !isUserCached)
                 m_permissionsCache->add(*user);
             return true;
         }
@@ -698,9 +698,7 @@ bool CLdapSecManager::authenticate(ISecUser* user)
     {
         user->setAuthenticateStatus(AS_AUTHENTICATED);
     }
-
-    //call LDAP to authenticate
-    if (AS_AUTHENTICATED != user->getAuthenticateStatus() && m_ldap_client->authenticate(*user))
+    else if (m_ldap_client->authenticate(*user)) //call LDAP to authenticate
         user->setAuthenticateStatus(AS_AUTHENTICATED);
 
     if (AS_AUTHENTICATED == user->getAuthenticateStatus())