Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19808 )
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]> --- 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 Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/19808 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8333f1d0c3b4e92ef8430c7ad91bb7da963aceb9 Gerrit-Change-Number: 19808 Gerrit-PatchSet: 3 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: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]>
