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.

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

Read bytes when fetching or uid_fetching. (workaround, do not merge!) #55

Closed
wants to merge 1 commit into from

Conversation

vandenoever
Copy link
Contributor

This returns raw bytes for each result as well as the id. This was previously #33

This is a first version of this patch. It {c,sh}ould have more error checking.

I've checked this patch with DavMail, an IMAP proxy.

This patch was rejected before but until there is a better patch, this one is useful when one wants to have structured responses. I'm using this personally, so I'm just opening this patch so people that also need this functionality now can use it.

The real fix should come in #28

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.2%) to 61.23% when pulling 4ca1d15 on vandenoever:fetch into 542ee15 on mattnenterprise:master.

@jonhoo jonhoo added the wontfix This will not be worked on label Nov 1, 2017
@vandenoever
Copy link
Contributor Author

Since rust-imap moved to imap-proto, this pull request is no longer relevant.
Instead of this patch, one can use run_command_and_read_response and then parse with imap-proto.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants