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

Fetch all headers for a list of messages #738

Closed
wants to merge 2 commits into from

Conversation

stormleoxia
Copy link

@stormleoxia stormleoxia commented Jul 30, 2018

Hi,

On the ImapFolderFetch.cs:

I've added the ability to fetch all headers for a list of UniqueId.
In order to do so, I remove the check on the length of fields HashSet and modified the tokens to fetch all headers in case the hashset is non null and empty.
I didn't want to change the API but maybe you'll feel that adding a new overload of Fetch methods is more appropriate.
Regards,

Fabien

@jstedfast
Copy link
Owner

This will break existing uses because the versions of the Fetch() method that do not take a HashSet<string> fields argument pass null into that method, so now any app that calls Fetch() not wanting any headers will fetch all headers.

@jstedfast jstedfast closed this in 899fd9b Jul 30, 2018
@jstedfast
Copy link
Owner

I've implemented this in an even simpler way by adding a MessageSummaryItems.Headers flag, which, if set, will request all headers.

@stormleoxia
Copy link
Author

Ok. Great. Thanks

@jstedfast jstedfast added the enhancement New feature or request label Nov 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants