ID: 14407
Updated by: [EMAIL PROTECTED]
Reported By: [EMAIL PROTECTED]
-Status: Open
+Status: Feedback
Bug Type: Mail related
Operating System: Windows 2000
PHP Version: 4.0CVS-2001-12-10
New Comment:
Can you post a unified diff against latest CVS HEAD to
[EMAIL PROTECTED] please ?
Previous Comments:
------------------------------------------------------------------------
[2001-12-10 12:27:12] [EMAIL PROTECTED]
The SendMail function that is used for Win32 platforms doesn't extract
the "From:" header address to use as for the SMTP MAIL FROM command.
As a result any mail bounces go to the "sendmail_from" php.ini option
instead of to the sender. This is handled correctly in the PostHeader
function (which doesn't add "From:" if it is already there), but not in
TSendMail. The patch for win32/sendmail.c below allows TSendMail to
use the e-mail from address if it is available, and default to the
sendmail_from setting otherwise.
124,128c124,142
<
< if (INI_STR("sendmail_from")){
< RPath = estrdup(INI_STR("sendmail_from"));
< } else {
< return 19;
---
>
> if (headers && strstr(headers, "From:")) {
> char *pos;
> pos = strstr(headers, "From:") + 5;
> while (pos && (*pos == ' '))
> pos++;
>
> if (pos && strlen(pos)) {
> RPath = estrdup(pos);
> RPath[strcspn(RPath, "\r\n")] = '\x0';
> }
> }
>
> if (!RPath) {
> if (INI_STR("sendmail_from")) {
> RPath = estrdup(INI_STR("sendmail_from"));
> } else {
> return 19;
> }
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=14407&edit=1
