Fix for #142

[mikejzx]

This fixes #142

Basically all that needed to be done was in the fetchBodySection method we had to use the headers from the entire fetched message rather than from the one passed to the function.

This is because ProtonMail's API for listing messages (/messages/<filter>...) does not seem to allow for listing messages with their ReplyTos field included - the field can only be retrieved with the entire message. The message given to fetchBodySection is retrieved using ListMessages, and thus is missing the ReplyTos header

Please let me know if I did this properly or not 😆

[emersion]

This sounds related to #129.

Contrary to the other PR, this one does increase the number of API calls.

We also probably need to also fetch the whole message when sending the ENVELOPE?

[mikejzx]

Yeah my bad, I should've tested this a little more. The PR works fine for my specific setup with offline mail storage - the slowdown is not noticeable for it to be a problem However I tried instead logging in directly to the hydroxide imap server using my MUA and found that trying to fetch mail headers takes a very long time with this patch applied, rendering it a little impractical :/

…

On Wednesday, Jan 06, 2021 at 21:26:14 +1100, Simon Ser ***@***.***> wrote: This sounds related to #129. Contrary to the other PR, this one does increase the number of API calls. We also probably need to also fetch the whole message when sending the ENVELOPE? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.*

[emersion]

Maybe we could check whether we can skip fetching the whole message by looking at BodyPartName.Fields. Not all email clients set it, though. Some may set it but still require a full fetch.