Hello Yida Wu, Jason Fehr, Pranav Lodha, Joe McDonnell, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22462
to look at the new patch set (#18).
Change subject: IMPALA-13687: Support shared secret key for cookies
......................................................................
IMPALA-13687: Support shared secret key for cookies
Adds support for a shared secret key across coordinators for cookie
validation. This is used with Thrift servers using HTTPS to support
sharing the same cookie across sessions with different coordinators.
Adds an inotify watcher on the cookie secret key to reload when it's
updated.
Testing:
- Adds new LdapHS2Test, LdapWebserverTest, and LdapImpalaShellTest
cases for shared cookie.
- Adds AuthenticationHash and AuthenticationHashFromFile unit tests.
Generated-by: Github Copilot (GPT-4.1)
Change-Id: Ie2e2345f771608069407e9dcf7ed4697fc0214e7
---
M be/src/rpc/auth-provider.h
M be/src/rpc/authentication.cc
M be/src/rpc/authentication.h
M be/src/util/openssl-util-test.cc
M be/src/util/openssl-util.cc
M be/src/util/openssl-util.h
M be/src/util/webserver.cc
M be/src/util/webserver.h
M fe/src/test/java/org/apache/impala/customcluster/LdapHS2Test.java
M fe/src/test/java/org/apache/impala/customcluster/LdapImpalaShellTest.java
M
fe/src/test/java/org/apache/impala/customcluster/LdapSearchBindImpalaShellTest.java
M
fe/src/test/java/org/apache/impala/customcluster/LdapSimpleBindImpalaShellTest.java
M fe/src/test/java/org/apache/impala/customcluster/LdapWebserverTest.java
M fe/src/test/java/org/apache/impala/testutil/TestUtils.java
M fe/src/test/java/org/apache/impala/testutil/WebClient.java
15 files changed, 568 insertions(+), 26 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/62/22462/18
--
To view, visit http://gerrit.cloudera.org:8080/22462
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie2e2345f771608069407e9dcf7ed4697fc0214e7
Gerrit-Change-Number: 22462
Gerrit-PatchSet: 18
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Pranav Lodha <[email protected]>
Gerrit-Reviewer: Yida Wu <[email protected]>
