DaanHoogland commented on a change in pull request #3694: Ldap fixes
URL: https://github.com/apache/cloudstack/pull/3694#discussion_r360311276
##########
File path:
plugins/user-authenticators/ldap/src/main/java/org/apache/cloudstack/ldap/LdapManagerImpl.java
##########
@@ -142,12 +142,15 @@ private LdapConfigurationResponse
addConfigurationInternal(final String hostname
public boolean canAuthenticate(final String principal, final String
password, final Long domainId) {
try {
// TODO return the right account for this user
- final LdapContext context =
_ldapContextFactory.createUserContext(principal, password,domainId);
+ final LdapContext context =
_ldapContextFactory.createUserContext(principal, password, domainId);
closeContext(context);
+ if(LOGGER.isTraceEnabled()) {
+ LOGGER.trace(String.format("User(%s) authenticated for
domain(%s)", principal, domainId));
+ }
return true;
- } catch (NamingException | IOException e) {
- s_logger.debug("Exception while doing an LDAP bind for user "+"
"+principal, e);
- s_logger.info("Failed to authenticate user: " + principal + ".
incorrect password.");
+ } catch (NamingException /* | AuthenticationException */ | IOException
e) {
Review comment:
extended as explanation
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
