Kaydet (Commit) 95ff7239 authored tarafından R David Murray's avatar R David Murray

#20013: don't raise socket error when selected mailbox deleted.

I'm checking this in without a test because not much of this code
is tested and I don't have time to work up the necessary extensions
to the existing test framework.

The patch itself was tested by the person who reported the bug.
üst feb68522
...@@ -1063,6 +1063,11 @@ class IMAP4: ...@@ -1063,6 +1063,11 @@ class IMAP4:
del self.tagged_commands[tag] del self.tagged_commands[tag]
return result return result
# If we've seen a BYE at this point, the socket will be
# closed, so report the BYE now.
self._check_bye()
# Some have reported "unexpected response" exceptions. # Some have reported "unexpected response" exceptions.
# Note that ignoring them here causes loops. # Note that ignoring them here causes loops.
# Instead, send me details of the unexpected response and # Instead, send me details of the unexpected response and
......
...@@ -24,6 +24,10 @@ Core and Builtins ...@@ -24,6 +24,10 @@ Core and Builtins
Library Library
------- -------
- Issue #20013: Some imap servers disconnect if the current mailbox is
deleted, and imaplib did not handle that case gracefully. Now it
handles the 'bye' correctly.
- Issue #20531: Revert 3.4 version of fix for #19063, and apply the 3.3 - Issue #20531: Revert 3.4 version of fix for #19063, and apply the 3.3
version. That is, do *not* raise an error if unicode is passed to version. That is, do *not* raise an error if unicode is passed to
email.message.Message.set_payload. email.message.Message.set_payload.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment