Bug #73485 [Nab]: mail() adds a lone CR or LF

From: Date: Thu, 24 Nov 2016 08:50:22 +0000
Subject: Bug #73485 [Nab]: mail() adds a lone CR or LF
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-205598@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=73485&edit=1

 ID:                 73485
 Updated by:         yohgaki@php.net
 Reported by:        sherwoodsports at gmail dot com
 Summary:            mail() adds a lone CR or LF
 Status:             Not a bug
 Type:               Bug
 Package:            Mail related
 Operating System:   All
 PHP Version:        7.0.X
 Block user comment: N
 Private report:     N

 New Comment:

OK. Back to original report.

Could you modify the test script and attach the modified test script (the phpt) so that the problem
can be seen as plain text output?

OS should not be "All". Are you using Windows or UNIX like OS? (Windows and UNIX uses
different code to send mails) Please specify what OS you are using.


Previous Comments:
------------------------------------------------------------------------
[2016-11-24 07:57:46] yohgaki@php.net

It trims unneeded chars at the first and end, detects malformed newlines, but it never add modified
headers by parsing addtional_headers.

If you have problem, please try to reproduce by modifying the phpt file that I've mentioned.

------------------------------------------------------------------------
[2016-11-24 07:51:59] yohgaki@php.net

Like I mentioned, you should provide "bug case", not creating new bug report.

PHP is not the only program modify mail header, so you should provide case that PHP produces
malformed mail headers.

BTW

To: <to@email.com>
X-PHP-Originating-Script: 0:test.php
 From: from@email.com
From: <from@email.com>  <--- PHP never adds this 

by 

mail("to@email.com", "Subject", "Message", "From:
from@email.com");


Current mail() accepts multiline header, i.e. it allows too long line folding by  "\r\n ",
"\r\n\t", but it never split lines automatically.

------------------------------------------------------------------------
[2016-11-24 05:55:13] mattcoz at yahoo dot com

I opened a new bug for my issue: https://bugs.php.net/bug.php?id=73507

------------------------------------------------------------------------
[2016-11-24 05:46:07] sherwoodsports at gmail dot com

Hold up...  the comments posted are not related to the original report.  Please do not identify this
bug with those comments then mark this bug as "not a bug" due to your analysis of the
issue identified in the comments.

------------------------------------------------------------------------
[2016-11-24 03:03:50] yohgaki@php.net

We have test cases for multiple headers by additional_headers parameter.
http://lxr.php.net/xref/PHP-7.0/ext/standard/tests/mail/mail_basic6.phpt

> mail("to@email.com", "Subject", "Message", "X-PHP-Fix:
> x\r\nFrom: from@email.com");

This shouldn't produce following headers.

> To: <to@email.com>
> X-PHP-Originating-Script: 0:test.php
>  From: from@email.com
> From: <from@email.com>

I suppose this is your MTA issue.

If you think this is a bug in PHP, try to add bug case to 
http://lxr.php.net/xref/PHP-7.0/ext/standard/tests/mail/mail_basic6.phpt

You can run the test in PHP source build like

TESTS=ext/standard/tests/mail/mail_basic6.phpt make test

Attach modified mail_basic6.phpt file and reopen this bug.

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=73485


--
Edit this bug report at https://bugs.php.net/bug.php?id=73485&edit=1


Thread (21 messages)

« previous php.bugs (#205598) next »