Norbert Luksa has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/20421 )

Change subject: IMPALA-12403: Fix Kerberos authentication when connecting with 
a proxy user that does not delegate another user
......................................................................

IMPALA-12403: Fix Kerberos authentication when connecting with a proxy
user that does not delegate another user

This change improves LDAP checking in the Hiveserver2 API where
previously LDAP filter checks were run on the user principal received
in Kerberos authentication, but after the modification they are run on
the short username taken from the Kerberos principal.

Tested with new custom cluster tests.

Change-Id: I0141cd341cda84ed40cd72f8a038c8d5a215e5e1
Reviewed-on: http://gerrit.cloudera.org:8080/20421
Tested-by: Impala Public Jenkins <[email protected]>
Reviewed-by: Norbert Luksa <[email protected]>
---
M be/src/rpc/authentication.cc
M be/src/rpc/thrift-server.h
M be/src/service/impala-hs2-server.cc
M be/src/transport/TSasl.cpp
M be/src/transport/TSasl.h
M be/src/transport/TSaslTransport.cpp
M be/src/transport/TSaslTransport.h
M be/src/util/auth-util.cc
M be/src/util/auth-util.h
M fe/src/test/java/org/apache/impala/customcluster/LdapImpalaShellTest.java
M 
fe/src/test/java/org/apache/impala/customcluster/LdapKerberosImpalaShellTest.java
M 
fe/src/test/java/org/apache/impala/customcluster/LdapSearchBindImpalaShellTest.java
12 files changed, 128 insertions(+), 22 deletions(-)

Approvals:
  Impala Public Jenkins: Verified
  Norbert Luksa: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/20421
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I0141cd341cda84ed40cd72f8a038c8d5a215e5e1
Gerrit-Change-Number: 20421
Gerrit-PatchSet: 8
Gerrit-Owner: Gergely Farkas <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Gergely Farkas <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Norbert Luksa <[email protected]>

Reply via email to