ID: 47983
User updated by: hannibal at astral dot lodz dot pl
Reported By: hannibal at astral dot lodz dot pl
Status: Open
Bug Type: Feature/Change Request
Operating System: Linux
PHP Version: 5.2.9
New Comment:
I don't know, why you are change Category - this is cleanly bug
report, not feature change request. I'm only suggest possible
solutions - you can resolve this in diffrent way. I'm just not want
to PHP send CRLF & LF mixed headers!
Previous Comments:
------------------------------------------------------------------------
[2009-04-16 09:05:08] hannibal at astral dot lodz dot pl
Description:
------------
Diffrent parts of mail function use LF some CRLF. This cause headers
to have mixed LF & CRLF endings. This can casue in some situation
broken mail delivery. PLEASE do not tell this is MTA problem. This is
also your concer to PROVIDE MTA standard compilant headers or at
least do this in consistent way.
Details:
ext/standard/mail.c (PHP 5.2.9) line 147
SKIP_LONG_HEADER_SEP(to_r, i);
This code assume that user provide CRLF line ending. If user provide
LF only, it is replace by spaces - so it cannot be used.
ext/standard/mail.c (PHP 5.2.9) line 273-278
fprintf(sendmail, "To: %s\n", to);
fprintf(sendmail, "Subject: %s\n", subject);
if (headers != NULL) {
fprintf(sendmail, "%s\n", headers);
}
fprintf(sendmail, "\n%s\n", message);
This cleanly show that headers added automatially by PHP are always
ended by LF only.
In case, where subject is multline, headers always have mixed CRLF
and LF. This cause Postfix to left CR in header lines that have it.
It can cause probles where buggy Qmail is recipent, it replaces alone
CR to empty line, which causes premature end of headers in recipent
mail program. Again this is also YOUR PROBLEM, MTA must only properly
handling RFC compilant data - when other is sended, behaviour can be
undefined.
There is a couple methods to fix it, more or less properly.
1. Add "\r" to fprintf - RFC compilant but probably broke many apps
which use LF only header endings.
2. Add support to LF ending to SKIP_LONG_HEADER_SEP - also quite
easy, but not necessery best.
3. Add prameter to mail function in which user provide desired EOL -
best some enum (LEGACY - current buggy, CRLF - RFC compilant, LF -
unix compilant)
4. Configuration parameter whith same values as above.
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=47983&edit=1
