commit e3eb9afb20a0535d44c203a4195db9822f8861fc
author Michael Smith <michael.smith@cloudera.com> Wed Jun 08 17:23:27 2022 -0700
committer Yuxuan 'fishy' Wang <fishywang@gmail.com> Fri Jun 10 08:41:25 2022 -0700
tree 22449e217b4ca184342dbc6e562cdd3db2fea45b
parent 4959a92385e2a6d7a4b8419784f85e5762c714cf

THRIFT-5595: Handle peek request error with SSLSocket

The update to TSocket in 01d53f483a7531ad4899b522060e8913dca309fb errors
for TSSLSocket with
```
ValueError: non-zero flags not allowed in calls to recv() on <class 'ssl.SSLSocket'>
```
Handles ValueError from ssl.SSLSocket to fix isOpen when using
TSSLSocket.

lib/py/test/test_socket.py

diff --git a/lib/py/test/test_socket.py b/lib/py/test/test_socket.py
index 95124dc..b732653 100644
--- a/lib/py/test/test_socket.py
+++ b/lib/py/test/test_socket.py
@@ -25,6 +25,7 @@
             acc.start()
 
             sock = TSocket(host="localhost", port=acc.port)
+            self.assertFalse(sock.isOpen())
             sock.open()
             sock.setTimeout(timeout)
 

lib/py/test/test_sslsocket.py

diff --git a/lib/py/test/test_sslsocket.py b/lib/py/test/test_sslsocket.py
index f4c87f1..3b77e39 100644
--- a/lib/py/test/test_sslsocket.py
+++ b/lib/py/test/test_sslsocket.py
@@ -158,7 +158,9 @@
     def _assert_connection_success(self, server, path=None, **client_args):
         with self._connectable_client(server, path=path, **client_args) as (acc, client):
             try:
+                self.assertFalse(client.isOpen())
                 client.open()
+                self.assertTrue(client.isOpen())
                 client.write(b"hello")
                 self.assertEqual(client.read(5), b"there")
                 self.assertTrue(acc.client is not None)