Skip to content

fix(imap): do not bubble up errors in IMAP candidate loop#8420

Merged
link2xt merged 2 commits into
mainfrom
link2xt/ouswksvxkspx
Jul 16, 2026
Merged

fix(imap): do not bubble up errors in IMAP candidate loop#8420
link2xt merged 2 commits into
mainfrom
link2xt/ouswksvxkspx

Conversation

@link2xt

@link2xt link2xt commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Network errors such as timeouts should not break out of the loop by bubbling up errors. There may be more candidates to try, breaking out of the loop skips trying them.

Comment thread src/imap.rs Outdated
@link2xt
link2xt force-pushed the link2xt/ouswksvxkspx branch 2 times, most recently from e48ebc2 to 9951265 Compare July 16, 2026 07:51
link2xt added 2 commits July 16, 2026 08:15
Network errors such as timeouts should not break out of the loop
by bubbling up errors. There may be more candidates to try,
breaking out of the loop skips trying them.
The loop is large, so having an explicit label
makes it easier to understand what the "continue" statement does.
@link2xt link2xt changed the title fix(imap): do not bubble up errors when failing to determine capabilities or enable compression fix(imap): do not bubble up errors in IMAP candidate loop Jul 16, 2026
@link2xt
link2xt force-pushed the link2xt/ouswksvxkspx branch from 9951265 to 0462e12 Compare July 16, 2026 08:16
@link2xt
link2xt merged commit 1a24bf6 into main Jul 16, 2026
56 of 57 checks passed
@link2xt
link2xt deleted the link2xt/ouswksvxkspx branch July 16, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants