Text parts not decoded

mfb - December 12, 2008 - 04:37
Project:Mailhandler
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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.

AttachmentSize
mailhandler-decode.patch1.31 KB

#1

z.stolar - December 25, 2008 - 12:27

Can you suggest a way to test this?

#2

mfb - December 26, 2008 - 00:04

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.

#3

z.stolar - January 5, 2009 - 20:16

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.

#4

mfb - January 5, 2009 - 20:23

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..

#5

mfb - January 6, 2009 - 22:44

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

#6

z.stolar - April 27, 2009 - 19:22
Status:needs review» fixed

Patch looks fine to me. applied.

#7

System Message - May 11, 2009 - 19:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.