Hi,

I've noticed a handful of issues with the mimemail_headers function:

* Return-Path has extra <> wrappers (reported here for 7.x http://drupal.org/node/1190144)
* Errors-To header is depreciated and causes problems with certain mail handlers (such as Amazon SES)
* From can contain an array (for example if null is passed as the sender to mimemail), which causes errors with preg_match

The attached patch resolves these issues against 6.x.1.dev.

CommentFileSizeAuthor
#2 mimemail_1270656_01.patch440 bytessgabe
mimemail.inc_.headers.patch1.02 KBgmania

Comments

sgabe’s picture

sgabe commented
Title: Multiple issues with function mimemail_headers » From header can be an array which causes errors with preg_match

#1275080: Remove the depricated Errors-To header is created to deal with the depreciated Errors-To header. This issue will be the one where we fix the problem with the From header.

sgabe’s picture

sgabe commented
StatusFileSize
new mimemail_1270656_01.patch440 bytes

I think we should prevent the From header to be an array. I could reproduce this with Rules actions when I only specified the sender name without an address.

sgabe’s picture

sgabe commented
Status: Needs review » Fixed

Committed for 6.x and already in 7.x.

Status: Fixed » Closed (fixed)

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