diff --git a/src/imap.rs b/src/imap.rs index c211b8b3d6..7505060d87 100644 --- a/src/imap.rs +++ b/src/imap.rs @@ -314,7 +314,8 @@ impl Imap { self.conn_backoff_ms = max(BACKOFF_MIN_MS, self.conn_backoff_ms); let login_params = prioritize_server_login_params(&context.sql, &self.lp, "imap").await?; - let mut first_error = None; + let mut first_connection_error = None; + let mut first_login_error = None; 'candidate: for lp in login_params { info!(context, "IMAP trying to connect to {}.", lp.connection); let connection_candidate = lp.connection.clone(); @@ -330,7 +331,7 @@ impl Imap { Ok(client) => client, Err(err) => { warn!(context, "{err:#}."); - first_error.get_or_insert(err); + first_connection_error.get_or_insert(err); continue 'candidate; } }; @@ -410,7 +411,7 @@ impl Imap { let message = stock_str::cannot_login(context, &imap_user); warn!(context, "IMAP failed to login: {err:#}."); - first_error.get_or_insert(format_err!("{message} ({err:#})")); + first_login_error.get_or_insert(format_err!("{message} ({err:#})")); // If it looks like the password is wrong, send a notification: let _lock = context.wrong_pw_warning_mutex.lock().await; @@ -446,7 +447,9 @@ impl Imap { } } - Err(first_error.unwrap_or_else(|| format_err!("No IMAP connection candidates provided"))) + Err(first_login_error + .or(first_connection_error) + .unwrap_or_else(|| format_err!("No IMAP connection candidates provided"))) } /// Prepare a new IMAP session.