tree a9aefb5eb6de9b3e458cd661264ec53b69b2e1d3
parent e26a0105061ad6eaa85188438ffa8f04a89500ca
author Bruce Schuchardt <bschuchardt@pivotal.io> 1573055694 -0800
committer Mark Hanson <mhanson@pivotal.io> 1573061053 -0800

Feature/geode 6661 (#4284)

* GEODE-6661 NioSslEngine has some problems in its ByteBuffer management

Reverting the change to use a temporary byte buffer for SSL handshakes.
At the end of a handshake the buffer may contain application data that
must be available for subsequent decryption.  In the case of TCPConduit
this is usually the "handshake" bytes transmitted for that package's
communications protocol.

Since we really need those bytes I've removed the option of expanding
the handshake buffer if it's smaller than the SSL session's required
packet size.  TCPConduit uses that figure to allocate the buffer so this
should be safe.  I've added a test for this.

* reverting investigative changes to test

* fix failing unit tests - adjusted buffer sizes

* reverted another set of investigative test changes