PHP mail() function triggers 'missing recipient' message

[geerlingguy]

I kept digging deeper and deeper (I really want to use mhsendmail instead of sSMTP for my Drupal dev environments!), and found that PHP's mail() function is the culprit here, in terms of how it uses sendmail...

To reproduce, install PHP on a server, set the sendmail_path ini directive to the path to the mhsendmail binary, then create a script somewhere with the following test:

<?php
$to = '[email protected]';
$subject = 'Test';
$message = 'Test message.';
$headers = 'From: [email protected]' . "\r\n" .
  'Reply-To: [email protected]' . "\r\n" .
  'X-Mailer: PHP/' . phpversion();
$result = mail($to, $subject, $message, $headers);
var_dump($result);
?>

If you run the script by accessing it through a webserver, check the webserver's error logs; you should get a message missing recipient, from mhsendmail.

The problem is that it seems mhsendmail is expecting the recipient as one of the command line args, but PHP's mail() function passes the recipient as a line in the header ('To: [recipient]'). See php-src: https://github.com/php/php-src/blob/fc33f52d8c25997dd0711de3e07d0dc260a18c11/ext/standard/mail.c#L356-L361

I think if you can make it so mhsendmail accepts the To address as the first line of stdin, it might work...

[ian-kent]

Thanks @geerlingguy, I'll have a look

[stucki]

I can confirm this problem.

The simple workaround is to provide a dummy email address in PHPs sendmail_path:

sendmail_path = "/usr/local/sbin/mhsendmail [email protected]"

The recipient will be overwritten when the "To:" header is added by PHPs mail() function.

[ian-kent]

Thanks, and sorry, haven't had time to look at it yet, I'll get a fix in asap!

[ashleyhindle]

👍

[stucki]

Thanks!

[geerlingguy]

Looks like this works, but could you tag a new release so we can download the binary for use in other projects? :)

[domodwyer]

+1 on the new binaries, I pull them using Chef for our development environment and just got bit by this!

[iefbr14]

The re for find To: doesn't work correctly
Here is a fix.

re := regexp.MustCompile("(?im)^To: (.*)\r*$")