Hello Impala Public Jenkins, Sailesh Mukil, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7606
to look at the new patch set (#6).
Change subject: IMPALA-5743: Support TLS version configuration for Thrift
servers
......................................................................
IMPALA-5743: Support TLS version configuration for Thrift servers
* Add --ssl_minimum_version which controls the minimum SSL/TLS version
that clients and servers will use when negotiating a secure
connection.
* Two kinds of version specification are allowed: 'TLSv1.1' enables
TLSv1.1 and all subsequent verisons. 'TLSv1.1_only' enables only
TLSv1.1. The latter is not exposed in user-facing text as it is
typically only used for testing.
* Handle case where platform may not support TLSv1.1 or v1.2 by checking
OpenSSL version number.
* Bump Thrift toolchain version to -p10.
Testing:
* New tests in thrift-server-test.cc. In particular, test all 36
configurations of client and server protocol versions, and ensure that
the expected successes or failures are seen.
Change-Id: I4c68a6c9658ddbfbe8025f2021fd5ed7a9dec5a5
---
M be/src/catalog/catalogd-main.cc
M be/src/rpc/thrift-client.cc
M be/src/rpc/thrift-client.h
M be/src/rpc/thrift-server-test.cc
M be/src/rpc/thrift-server.cc
M be/src/rpc/thrift-server.h
M be/src/service/impala-server.cc
M be/src/statestore/statestore-subscriber.cc
M be/src/statestore/statestored-main.cc
M bin/impala-config.sh
10 files changed, 215 insertions(+), 26 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/06/7606/6
--
To view, visit http://gerrit.cloudera.org:8080/7606
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4c68a6c9658ddbfbe8025f2021fd5ed7a9dec5a5
Gerrit-PatchSet: 6
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Henry Robinson <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Sailesh Mukil <[email protected]>
