Hello Csaba Ringhofer, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23519
to look at the new patch set (#4).
Change subject: IMPALA-14452: Fix impala-shell SSL with Python 3.12
......................................................................
IMPALA-14452: Fix impala-shell SSL with Python 3.12
Removes deprecated ImpalaHttpClient constructor that supported port and
path as it has been deprecated since at least 2020 and appears unused.
Removes cert_file and key_file as they were also never used, and if
required must now be passed in via ssl_context.
Updates TSSLSocket fixes for Thrift 0.16 and Python 3.12. _validate_cert
was removed by Thrift 0.16, but everything worked because Thrift used
ssl.match_hostname instead. With Python 3.12 we need to bypass matching
that fails because ssl.match_hostname no longer exists and let OpenSSL
handle verification.
Switches to ssl.PROTOCOL_TLS to future-proof removal. Python 2.7.13 and
3.6 are minimum required versions; both have been EOL for several years.
Testing: ran tests/custom_cluster/test_client_ssl.py with Python 3.12.
Change-Id: I046a9010ac4cb1f7d705935054b306cddaf8bdc7
---
M setup.cfg
M shell/impala_shell/ImpalaHttpClient.py
A shell/impala_shell/TSSLSocketWithFixes.py
D shell/impala_shell/TSSLSocketWithWildcardSAN.py
M shell/impala_shell/impala_client.py
M shell/impala_shell/impala_shell.py
6 files changed, 77 insertions(+), 218 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/19/23519/4
--
To view, visit http://gerrit.cloudera.org:8080/23519
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I046a9010ac4cb1f7d705935054b306cddaf8bdc7
Gerrit-Change-Number: 23519
Gerrit-PatchSet: 4
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
