I found that for email messages with a number of base64-encoded text parts, the text parts are not decoded by mailhandler. Here's a patch that resolves the problem for me.

I have not yet tested this on a large number of e-mails. But if I find any problems with it I'll report back here.

CommentFileSizeAuthor
mailhandler-decode.patch1.31 KBmfb

Comments

z.stolar’s picture

z.stolar commented

Can you suggest a way to test this?

mfb’s picture

mfb
they or he
commented

To test the bug you just need a message with multiple base64-encoded text parts. I can forward you such a message if you don't have any way to generate it and you give me an e-mail address.

z.stolar’s picture

z.stolar commented

Sure - I'd love to learn what the problem looks like. My address is zohar at linnovate dot net.
However - please feel free to commit any patch you judge adequate and ready. I hate being a bottleneck and I fully trust your judgment.

mfb’s picture

mfb
they or he
commented

We could really use a test suite for this module, complete with a set of sample messages. That would make me feel more confident about committing such a patch. I'm not sure how, when an IMAP server is needed for testing..

mfb’s picture

mfb
they or he
commented

FYI I just bounced a sample message to you, the original message structure should be intact (I used mutt, my favourite mail client ;)

z.stolar’s picture

z.stolar commented
Status: Needs review » Fixed

Patch looks fine to me. applied.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.