Merge branch 'pr/10'

[MRM-1923] merge pr
This commit is contained in:
olivier lamy 2016-09-13 10:55:00 +10:00
commit 11b2cdbae0

View File

@ -122,18 +122,25 @@ public AuthenticationResult authenticate( AuthenticationDataSource s )
results = context.search( mapper.getUserBaseDn(), filter, ctls );
log.debug( "Found user '{}': {}", source.getUsername(), results.hasMoreElements() );
boolean moreElements = results.hasMoreElements();
if ( results.hasMoreElements() )
log.debug( "Found user '{}': {}", source.getUsername(), moreElements );
if ( moreElements )
{
SearchResult result = results.nextElement();
try {
SearchResult result = results.nextElement();
userDn = result.getNameInNamespace();
userDn = result.getNameInNamespace();
log.debug( "Adding userDn {} for user {} to the cache..", userDn, source.getUsername() );
log.debug("Adding userDn {} for user {} to the cache..", userDn, source.getUsername());
// REDBACK-289/MRM-1488 cache the ldap user's userDn to lessen calls to ldap server
ldapCacheService.addLdapUserDn( source.getUsername(), userDn );
// REDBACK-289/MRM-1488 cache the ldap user's userDn to lessen calls to ldap server
ldapCacheService.addLdapUserDn(source.getUsername(), userDn);
} catch (Exception e) {
log.error("Error occured on LDAP result retrieval: {}, {}", userDn, e.getMessage());
return new AuthenticationResult( false, source.getUsername(), e);
}
}
else
{