Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17198


Change subject: [security] enhance tls_handshake test
......................................................................

[security] enhance tls_handshake test

This patch adds extra verification step into the TestHandshakeSequence
scenario of TestTlsHandshake.  That's to assert that both parties can
send and receive application data using OpenSSL API (i.e. SSL_Read() and
SSL_Write() function) once TlsHandshake::Continue() returns Status::OK.
In addition, the newly introduced utility methods in TestTlsHandshake
are useful for a follow-up changelist as well.

In addition, I cleaned up code in {client,server}_negotiation.cc
and in tls_context.cc.

Change-Id: Ie77d1f35e12fe74be9f4022e8cd7087500052743
---
M src/kudu/rpc/client_negotiation.cc
M src/kudu/rpc/server_negotiation.cc
M src/kudu/security/tls_context.cc
M src/kudu/security/tls_handshake-test.cc
M src/kudu/security/tls_handshake.h
5 files changed, 98 insertions(+), 20 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie77d1f35e12fe74be9f4022e8cd7087500052743
Gerrit-Change-Number: 17198
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to