Jason Fehr has uploaded a new patch set (#7). ( 
http://gerrit.cloudera.org:8080/23300 )

Change subject: IMPALA-14312: Fix Issues with ssl_minimum_version Flag Validator
......................................................................

IMPALA-14312: Fix Issues with ssl_minimum_version Flag Validator

The validator for the ssl_minimum_version flag has several issues
that are fixed.

Allows flag to be empty as long as both internal and external TLS is
not configured.

Fixes allowed value for TLS v1 to be tlsv1 instead of the incorrect
value tlsv1.0.

Removes "tlsv1.3" as an allowed value since Thrift does not support
that value as the minimum TLS version.

Testing accomplished by new ctests and manual testing.

Change-Id: I6493852b581e26c203b6b46b97be76100bcc534b
---
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-test.cc
M be/src/service/impala-server.cc
M be/src/util/openssl-util.cc
M be/src/util/openssl-util.h
7 files changed, 164 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/00/23300/7
--
To view, visit http://gerrit.cloudera.org:8080/23300
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6493852b581e26c203b6b46b97be76100bcc534b
Gerrit-Change-Number: 23300
Gerrit-PatchSet: 7
Gerrit-Owner: Jason Fehr <jf...@cloudera.com>
Gerrit-Reviewer: Abhishek Rawat <ara...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>

Reply via email to