Hello Alexey Serbin, Abhishek Chennaka,

I'd like you to do a code review. Please visit

    http://gerrit.cloudera.org:8080/23253

to review the following change.


Change subject: [webserver] Add support for TLS 1.3
......................................................................

[webserver] Add support for TLS 1.3

This commit patches Squeasel to honor the minimum TLS version even if
TLSv1.3 is set and adds a new ciphersuites option in addition to
ciphers, which is used for TLS 1.3.

I added a new test that requires TLS 1.3, but the new ciphersuites
option was tested only manually with openssl s_client, as curl doesn't
seem to support requiring specific ciphers.

Change-Id: Iba21d62e22962c782ff8013d805b31ff058d9245
Reviewed-on: http://gerrit.cloudera.org:8080/23149
Reviewed-by: Alexey Serbin <[email protected]>
Tested-by: Attila Bukor <[email protected]>
(cherry picked from commit f6fbcde39f48fc8ae119a6ecce03a8a4254d8a02)
---
M src/kudu/server/webserver-test.cc
M src/kudu/server/webserver.cc
M src/kudu/server/webserver_options.cc
M src/kudu/server/webserver_options.h
M src/kudu/util/curl_util.cc
M src/kudu/util/curl_util.h
M thirdparty/download-thirdparty.sh
A thirdparty/patches/squeasel-tls-min-version.patch
8 files changed, 205 insertions(+), 5 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.18.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iba21d62e22962c782ff8013d805b31ff058d9245
Gerrit-Change-Number: 23253
Gerrit-PatchSet: 1
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>

Reply via email to