Hello Todd Lipcon, Alexey Serbin,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/6042

to review the following change.

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
---
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(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/42/6042/1
-- 
To view, visit http://gerrit.cloudera.org:8080/6042
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

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

Reply via email to