Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19979 )
Change subject: [client] KUDU-3472 C++ client API to import JWT ...................................................................... [client] KUDU-3472 C++ client API to import JWT This patch addresses add C++ client API to specify a JWT for a newly built C++ client instance. It's an improvement in the usability of the Kudu client API since with this new extra method the users don't need to mess with the serialization of the authn credentials. The new functionality is covered by already existing tests that have been updated to start using the new API. More tests might be added to cover other aspects of the JWT-related functionality, but these are not directly related to this API change, so we can take care of them in a separate patch. Change-Id: I8333f1d0c3b4e92ef8430c7ad91bb7da963aceb9 Reviewed-on: http://gerrit.cloudera.org:8080/19808 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Alexey Serbin <[email protected]> (cherry picked from commit e0ef4a861bc5597b71a1a0ebdb88bde76d260a66) Reviewed-on: http://gerrit.cloudera.org:8080/19979 Tested-by: Abhishek Chennaka <[email protected]> --- M src/kudu/client/client.cc M src/kudu/client/client.h M src/kudu/client/client_builder-internal.h M src/kudu/integration-tests/security-itest.cc 4 files changed, 87 insertions(+), 60 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/19979 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I8333f1d0c3b4e92ef8430c7ad91bb7da963aceb9 Gerrit-Change-Number: 19979 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]>
