Tamas Mate has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17794
Change subject: IMPALA-10822: Allow multiple group retuns for LDAP Search ...................................................................... IMPALA-10822: Allow multiple group retuns for LDAP Search This commit updates the LdapSearch method to allow multiple results. LdapSearch method was written generally and used for both user and group searches. While it is reasonable to fail authentication requests where the LDAP Search returns more than one user this should not be the case for group searches, as a user can be part of multiple groups matching the search criteria. This commit moves the logic that checks the number of results to the specific user and group search methods. Testing: - Updated the unit test to cover this use-case Change-Id: I27eaf717985207e36d4fa47a2af757d06544474e --- M be/src/util/ldap-search-bind.cc M be/src/util/ldap-simple-bind.cc M be/src/util/ldap-util.cc M be/src/util/ldap-util.h M fe/src/test/java/org/apache/impala/customcluster/LdapSearchBindImpalaShellTest.java M fe/src/test/resources/users.ldif 6 files changed, 42 insertions(+), 34 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/94/17794/2 -- To view, visit http://gerrit.cloudera.org:8080/17794 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I27eaf717985207e36d4fa47a2af757d06544474e Gerrit-Change-Number: 17794 Gerrit-PatchSet: 2 Gerrit-Owner: Tamas Mate <tm...@cloudera.com>
