Tamas Mate has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19561 )

Change subject: IMPALA-11726: Allow LDAP user and group filter when Kerberos is 
enabled
......................................................................


Patch Set 6:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/19561/6/be/src/rpc/authentication.cc
File be/src/rpc/authentication.cc:

http://gerrit.cloudera.org:8080/#/c/19561/6/be/src/rpc/authentication.cc@988
PS6, Line 988: if (FLAGS_enable_ldap_auth || IsKerberosEnabled()) {
> Yes, if both LDAP and Kerberos authentication mode are enabled, the followi
Thank you for the detailed answer. Just trying to better understand the 
execution flow, does this mean that all three callback methods are called and 
should return with true to successfully authenticate?

I.e: if all callabacks are invoked, does that mean that 
SaslKerberosAuthorizeExternal does not need to check LDAP because the next 
callback LDAP_EXT_CALLBACKS does it?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If3ca9c4ff8a17167e5233afabdd14c948edb46de
Gerrit-Change-Number: 19561
Gerrit-PatchSet: 6
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: Tamas Mate <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Comment-Date: Tue, 07 Mar 2023 09:38:54 +0000
Gerrit-HasComments: Yes

Reply via email to