Fetch can hang when IMAP server aborts about mailkit HOT 9 CLOSED

I can possibly improve cancellation such as making it so that cancel will close the socket, thus causing the Stream.Read() to be aborted, but I'm not sure there's any other way to work around this.

Thanks for the bug report, I'll try to look into this soon.

from mailkit.

I think my bug description wasn't correct. When I said "hang" I meant that's what it looks like from the caller of Fetch(). It's not actually blocking in System.Net.Security.SslStream.Read but rather spinning in the methods mentioned (until canceled). So I think fixing this should be possible by making sure all occurrences of EOF are handled.

from mailkit.

Oh, thanks for the clarification.

from mailkit.

Could you test to see if this patch fixes the problem for you?

from mailkit.

Yes, that fixes it (exception gets thrown). Thanks.

from mailkit.

Awesome. I'll try to make similar fixes for SMTP and POP3 later today, as they probably suffer the same sort of problem.

from mailkit.

@mganss would you be willing to send me the first few lines of a log connecting to Courier IMAP? I mostly care about the greeting, but the CAPABILITIES would also be nice. Thanks!

from mailkit.

S: * OK [CAPABILITY IMAP4rev1 UIDPLUS CHILDREN NAMESPACE THREAD=ORDEREDSUBJECT THREAD=REFERENCES SORT QUOTA IDLE ACL ACL2=UNION STARTTLS] Courier-IMAP ready. Copyright 1998-2011 Double Precision, Inc.  See COPYING for distribution information.
C: A00000000 STARTTLS
S: A00000000 OK Begin SSL/TLS negotiation now.
C: A00000001 CAPABILITY
S: * CAPABILITY IMAP4rev1 UIDPLUS CHILDREN NAMESPACE THREAD=ORDEREDSUBJECT THREAD=REFERENCES SORT QUOTA IDLE AUTH=PLAIN ACL ACL2=UNION
S: A00000001 OK CAPABILITY completed

Is that enough?

from mailkit.

@mganss yes, that is perfect! Thank you!

from mailkit.