Repository: ambari Updated Branches: refs/heads/feature-branch-AMBARI-21307 25fa701c3 -> 735ff7b79 (forced update)
AMBARI-21307 fixed test and implementation Project: http://git-wip-us.apache.org/repos/asf/ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/735ff7b7 Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/735ff7b7 Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/735ff7b7 Branch: refs/heads/feature-branch-AMBARI-21307 Commit: 735ff7b79d98124fdeccd514144f8b608dc0c906 Parents: 7800777 Author: lpuskas <[email protected]> Authored: Tue Oct 24 15:29:32 2017 +0200 Committer: lpuskas <[email protected]> Committed: Thu Oct 26 13:35:09 2017 +0200 ---------------------------------------------------------------------- .../server/ldap/service/ads/DefaultLdapConfigurationService.java | 2 +- .../org/apache/ambari/server/ldap/LdapModuleFunctionalTest.java | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ambari/blob/735ff7b7/ambari-server/src/main/java/org/apache/ambari/server/ldap/service/ads/DefaultLdapConfigurationService.java ---------------------------------------------------------------------- diff --git a/ambari-server/src/main/java/org/apache/ambari/server/ldap/service/ads/DefaultLdapConfigurationService.java b/ambari-server/src/main/java/org/apache/ambari/server/ldap/service/ads/DefaultLdapConfigurationService.java index d80a636..3f6995c 100644 --- a/ambari-server/src/main/java/org/apache/ambari/server/ldap/service/ads/DefaultLdapConfigurationService.java +++ b/ambari-server/src/main/java/org/apache/ambari/server/ldap/service/ads/DefaultLdapConfigurationService.java @@ -65,7 +65,7 @@ public class DefaultLdapConfigurationService implements LdapConfigurationService Boolean isConnected = ldapConnectionTemplate.execute(new ConnectionCallback<Boolean>() { @Override public Boolean doWithConnection(LdapConnection connection) throws LdapException { - return connection.isConnected(); + return connection.isConnected() && connection.isAuthenticated(); } }); http://git-wip-us.apache.org/repos/asf/ambari/blob/735ff7b7/ambari-server/src/test/java/org/apache/ambari/server/ldap/LdapModuleFunctionalTest.java ---------------------------------------------------------------------- diff --git a/ambari-server/src/test/java/org/apache/ambari/server/ldap/LdapModuleFunctionalTest.java b/ambari-server/src/test/java/org/apache/ambari/server/ldap/LdapModuleFunctionalTest.java index c43e06c..30f5e22 100644 --- a/ambari-server/src/test/java/org/apache/ambari/server/ldap/LdapModuleFunctionalTest.java +++ b/ambari-server/src/test/java/org/apache/ambari/server/ldap/LdapModuleFunctionalTest.java @@ -93,8 +93,6 @@ public class LdapModuleFunctionalTest { Boolean success = template.execute(new ConnectionCallback<Boolean>() { @Override public Boolean doWithConnection(LdapConnection connection) throws LdapException { - connection.unBind(); - connection.bind(ldapConfiguration.bindDn(), ldapConfiguration.bindPassword()); return connection.isConnected() && connection.isAuthenticated(); }
