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>