[Python-checkins] cpython (merge 3.5 -> default): Issue #23779: imaplib raises TypeError if authenticator tries to abort.
robert.collins
python-checkins at python.org
Thu Jul 30 23:03:40 CEST 2015
https://hg.python.org/cpython/rev/0879f2c53289
changeset: 97157:0879f2c53289
parent: 97154:f05fc1a16e53
parent: 97156:b6f04b9d8c12
user: Robert Collins <rbtcollins at hp.com>
date: Fri Jul 31 09:02:09 2015 +1200
summary:
Issue #23779: imaplib raises TypeError if authenticator tries to abort.
Patch from Craig Holmquist.
files:
Lib/imaplib.py | 2 +-
Lib/test/test_imaplib.py | 20 ++++++++++++++++++++
Misc/ACKS | 1 +
Misc/NEWS | 3 +++
4 files changed, 25 insertions(+), 1 deletions(-)
diff --git a/Lib/imaplib.py b/Lib/imaplib.py
--- a/Lib/imaplib.py
+++ b/Lib/imaplib.py
@@ -1353,7 +1353,7 @@
def process(self, data):
ret = self.mech(self.decode(data))
if ret is None:
- return '*' # Abort conversation
+ return b'*' # Abort conversation
return self.encode(ret)
def encode(self, inp):
diff --git a/Lib/test/test_imaplib.py b/Lib/test/test_imaplib.py
--- a/Lib/test/test_imaplib.py
+++ b/Lib/test/test_imaplib.py
@@ -419,6 +419,26 @@
ret, data = client.login_cram_md5("tim", b"tanstaaftanstaaf")
self.assertEqual(ret, "OK")
+
+ @reap_threads
+ def test_aborted_authentication(self):
+
+ class MyServer(SimpleIMAPHandler):
+
+ def cmd_AUTHENTICATE(self, tag, args):
+ self._send_textline('+')
+ self.response = yield
+
+ if self.response == b'*\r\n':
+ self._send_tagged(tag, 'NO', '[AUTHENTICATIONFAILED] aborted')
+ else:
+ self._send_tagged(tag, 'OK', 'MYAUTH successful')
+
+ with self.reaped_pair(MyServer) as (server, client):
+ with self.assertRaises(imaplib.IMAP4.error):
+ code, data = client.authenticate('MYAUTH', lambda x: None)
+
+
def test_linetoolong(self):
class TooLongHandler(SimpleIMAPHandler):
def handle(self):
diff --git a/Misc/ACKS b/Misc/ACKS
--- a/Misc/ACKS
+++ b/Misc/ACKS
@@ -603,6 +603,7 @@
Shane Holloway
Rune Holm
Thomas Holmes
+Craig Holmquist
Philip Homburg
Naofumi Honda
Jeffrey Honig
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -13,6 +13,9 @@
Library
-------
+- Issue #23779: imaplib raises TypeError if authenticator tries to abort.
+ Patch from Craig Holmquist.
+
- Issue #24360: Improve __repr__ of argparse.Namespace() for invalid
identifiers. Patch by Matthias Bussonnier.
--
Repository URL: https://hg.python.org/cpython
More information about the Python-checkins
mailing list