Mail Header Fields and Newlines: separators vs. terminators

Discussions and requests related to new CMSimple features, plugins, templates etc. and how to develop.
Please don't ask for support at this forums!
Post Reply
cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Mail Header Fields and Newlines: separators vs. terminators

Post by cmb » Wed Apr 23, 2014 1:30 pm

Hello Community,

a while ago I stumbled upon the PHP manual entry for mail() (emphasis mine):
Multiple extra headers should be separated with a CRLF (\r\n).
However, the built-in mailform terminates multiple extra headers with CRLF (i.e. an additional CRLF is appended). I've tested with and without on PHP 5.4.19, and it made no difference to what PHP sent to the MTA, but we may consider to change the current behavior nonetheless to comply with the specification (assuming the documentation is correctly reflecting the specification).

Christoph
Christoph M. Becker – Plugins for CMSimple_XH

manu
Posts: 1086
Joined: Wed Jun 04, 2008 12:05 pm
Location: St. Gallen - Schweiz
Contact:

Re: Mail Header Fields and Newlines: separators vs. terminat

Post by manu » Wed Jun 25, 2014 1:42 pm

cmb wrote:However, the built-in mailform terminates multiple extra headers with CRLF (i.e. an additional CRLF is appended) ...
only if the additional $header parameter of XH_Mailform::sendMail() is empty...
I had quite a while to get the point...
+1

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: Mail Header Fields and Newlines: separators vs. terminat

Post by cmb » Tue Aug 12, 2014 6:14 pm

Done (r1337).
Christoph M. Becker – Plugins for CMSimple_XH

Post Reply