Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Discarded EXPUNGE/EXISTS responses #72

Closed
jonhoo opened this issue Nov 23, 2018 · 2 comments
Closed

Discarded EXPUNGE/EXISTS responses #72

jonhoo opened this issue Nov 23, 2018 · 2 comments
Labels
enhancement New feature or request help wanted Extra attention is needed tombstone A reference to an old issue on mattnenterprise/rust-imap
Milestone
3.0.0

Comments

@jonhoo
Copy link
Owner

jonhoo commented Nov 23, 2018 •
edited
Loading

When the wait functions return on an IdleHandle, the response doesn't contain the list of untagged EXPUNGE and EXISTS responses that have occurred. This means we don't necessarily know which sequence ids correspond to which messages anymore. E.g, we may be able to determine that a message was deleted because the exists parameter on the mailbox was decremented, but we don't know if that was the message with sequence id 1 or 10,000. This means that we have to maintain a record of the UID list and run unnecessary UID searches sometimes.

The wait functions currently return a Result<()>. They could be updated to return a list of EXPUNGE and EXISTS responses.

See the original issue here: mattnenterprise/rust-imap#72
@jonhoo jonhoo added enhancement New feature or request help wanted Extra attention is needed tombstone A reference to an old issue on mattnenterprise/rust-imap labels Nov 23, 2018
@jonhoo jonhoo mentioned this issue Nov 23, 2018
Closed
@jonhoo jonhoo added this to the 3.0.0 milestone Apr 20, 2021
@jonhoo
Copy link
Owner Author

jonhoo commented Apr 20, 2021

I think this is perhaps now solved with #186 @mordak?

@mordak
Copy link
Contributor

mordak commented Apr 21, 2021

I think this is perhaps now solved with #186 @mordak?

I think so. If the caller wants the responses then they can get them. EXPUNGE and EXISTS are available.

@jonhoo jonhoo closed this as completed Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request help wanted Extra attention is needed tombstone A reference to an old issue on mattnenterprise/rust-imap
Projects
None yet
Milestone
3.0.0
Development

No branches or pull requests
2 participants
@jonhoo @mordak