commit a72ffbea4d05586d39b507e5cd6451086fc4f763
author Tristen Georgiou <tristen.georgiou@gmail.com> Sat Feb 20 07:28:47 2016 +0900
committer Nobuaki Sukegawa <nsuke@apache.org> Mon Feb 22 23:24:31 2016 +0900
tree 12c3eb630025d13e13e0dcfaea32ad90f81dbb48
parent 33de26cf658450513ef5731ac55b635addb4d43d

THRIFT-3649 Python TSaslClientTransport initializes TTransportException incorrectly
Client: Python
Patch: Tristen Georgiou modified by Nobuaki Sukegawa

This closes #879

lib/py/src/transport/TTransport.py

diff --git a/lib/py/src/transport/TTransport.py b/lib/py/src/transport/TTransport.py
index 6669891..5283fce 100644
--- a/lib/py/src/transport/TTransport.py
+++ b/lib/py/src/transport/TTransport.py
@@ -382,13 +382,17 @@
                 self.send_sasl_msg(self.OK, self.sasl.process(challenge))
             elif status == self.COMPLETE:
                 if not self.sasl.complete:
-                    raise TTransportException("The server erroneously indicated "
-                                              "that SASL negotiation was complete")
+                    raise TTransportException(
+                        TTransportException.NOT_OPEN,
+                        "The server erroneously indicated "
+                        "that SASL negotiation was complete")
                 else:
                     break
             else:
-                raise TTransportException("Bad SASL negotiation status: %d (%s)"
-                                          % (status, challenge))
+                raise TTransportException(
+                    TTransportException.NOT_OPEN,
+                    "Bad SASL negotiation status: %d (%s)"
+                    % (status, challenge))
 
     def send_sasl_msg(self, status, body):
         header = pack(">BI", status, len(body))