Dan Burkert has submitted this change and it was merged.

Change subject: [security] add TLS protocol and cipher suite to negotiation 
trace
......................................................................


[security] add TLS protocol and cipher suite to negotiation trace

No tests, but I manually verified the output looks good:

0216 14:14:45.214356 (+    48us) server_negotiation.cc:484] Negotiated TLSv1.2 
with cipher suite AES256-GCM-SHA384

Change-Id: Ie1bf0d4cb8b683011e122a87b1856d718ba62331
Reviewed-on: http://gerrit.cloudera.org:8080/6042
Tested-by: Kudu Jenkins
Reviewed-by: Dan Burkert <danburk...@apache.org>
---
M src/kudu/rpc/client_negotiation.cc
M src/kudu/rpc/server_negotiation.cc
M src/kudu/security/tls_handshake.cc
M src/kudu/security/tls_handshake.h
4 files changed, 28 insertions(+), 2 deletions(-)

Approvals:
  Dan Burkert: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie1bf0d4cb8b683011e122a87b1856d718ba62331
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to