Skip to content

Conversation

DavidEichmann
Copy link
Contributor

The timeout was incorrectly set to 24 hours. This has been changed to 29 minutes to reflect the documentation on Handle struct and the reccomendation in RFC 2177.

The timeout was incorrectly set to 24 hours. This has been changed to
29 minutes to reflect the documentation on Handle struct and the
reccomendation in RFC 2177.
@DavidEichmann
Copy link
Contributor Author

DavidEichmann commented Sep 24, 2025

I have observed in my production system that Handle::wait(..) returns Ok(..) after about 30 minutes, which is immediatly followed by a Handle::done() that returns a connection lost error. I suspect that Handle::wait() returns Ok(ManualInterupt) when the server drops the connection due to a timeout. As a separate issue, we should consider if Handle::wait() should return Ok(ManualInterupt) or Err(/* connection lost */).

@link2xt link2xt merged commit 15f80ea into chatmail:main Sep 24, 2025
11 checks passed
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