From php mail() docs: The additional_parameters parameter is disabled in safe_mode and the mail() function will expose a warning message and return FALSE when used.

So on line 202, the fifth parameter should not be added at all if safe mode is enabled; it will not only be ignored, the mail will fail.

Comments

sgabe’s picture

sgabe commented

@pillarsdotnet: Just a note, but this is exactly what I meant when I said that "even the bugs are the same". See #261028: SMTP Return-Path Setting in Mime Mail's issue queue. :)

pillarsdotnet’s picture

pillarsdotnet commented
Version: 7.x-2.8 » 7.x-2.x-dev

Weird.

In most parts of PHP, passing NULL to a parameter is the same as not passing that parameter at all. But I guess this is just one more case of PHP being inconsistent with itself.

Fixed in -dev; will roll a new release shortly.

pillarsdotnet’s picture

pillarsdotnet commented
Assigned: Unassigned » pillarsdotnet
Status: Active » Fixed

Fixed in 7.x-2.48 release.

@sgabe: The difference is that bugs reported in the HTML Mail issue queue get fixed in a timely manner.

sgabe’s picture

sgabe commented

I just want to remind you that for more than one year nothing happened in HTML Mail's issue queue. It's nice that you have time now to work on the module, but these are big words after 3 months of maintaining. :)

pillarsdotnet’s picture

pillarsdotnet commented

I just want to remind you that for more than one year nothing happened in HTML Mail's issue queue.

I wasn't involved with the project at that point.

Status: Fixed » Closed (fixed)

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