Dan Burkert has submitted this change and it was merged. Change subject: [rpc] Assume existence of TlsContext during negotiation ......................................................................
[rpc] Assume existence of TlsContext during negotiation This changes the client and server RPC negotiatiators to always assume the existence of a TlsContext for the messenger, and check whether there are valid certs in the context in order to decide whether to use TLS encryption. This will simplify authentication negotiation down the line. Change-Id: I806ca360bcd168b1ab075a4c7143436a7ccbba06 Reviewed-on: http://gerrit.cloudera.org:8080/5946 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/rpc/client_negotiation.cc M src/kudu/rpc/client_negotiation.h M src/kudu/rpc/messenger.cc M src/kudu/rpc/messenger.h M src/kudu/rpc/negotiation-test.cc M src/kudu/rpc/negotiation.cc M src/kudu/rpc/server_negotiation.cc M src/kudu/rpc/server_negotiation.h 8 files changed, 74 insertions(+), 66 deletions(-) Approvals: Todd Lipcon: Looks good to me, but someone else must approve Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5946 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I806ca360bcd168b1ab075a4c7143436a7ccbba06 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
