Skip to content

Commit c841cae

Browse files
committed
✅ Fix FakeServer error handling on LOGOUT
I haven't traced down what's _causing_ the error. It seems like the socket is closed prematurely, so probably a race condition? At any rate, it should be safe to simply warn about the exception rather than crash.
1 parent 7a37e97 commit c841cae

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/net/imap/fake_server/command_router.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ def handler_for(command)
6060
rescue IOError
6161
# TODO: fix whatever is causing this!
6262
warn "connection issue after bye but before LOGOUT could complete"
63-
if $!.respond_to :detailed_message
63+
if $!.respond_to? :detailed_message
6464
warn $!.detailed_message highlight: true, order: :bottom
6565
else
6666
warn $!.full_message highlight: true, order: :bottom

0 commit comments

Comments
 (0)