Discussions and requests related to new CMSimple features, plugins, templates etc. and how to develop.
Please don't ask for support at this forums!
-
cmb
- Posts: 14225
- Joined: Tue Jun 21, 2011 11:04 am
- Location: Bingen, RLP, DE
-
Contact:
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
-
manu
- Posts: 1086
- Joined: Wed Jun 04, 2008 12:05 pm
- Location: St. Gallen - Schweiz
-
Contact:
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:
Post
by cmb » Tue Aug 12, 2014 6:14 pm
Done (r1337).