#47983 [Opn]: schizophrenic behaviour of adding EOL indicator in mail()

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