Fixed a bug where mail messages were malformed when using the Sendmail #3757

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

stewartlord commented Feb 12, 2013

transport on Unix systems with certain versions of sendmail (e.g.
Postfix < 2.9). Specifically, message headers spilled into the body
and the body text was double-spaced.

The source of the problem was the use of CRLF as the EOL sequence
when sending messages via PHP's built-in mail() function on Unix
systems. Despite what the PHP manual says, LF should be used as the
sole EOL character on Unix. This is because mail() is a wrapper for
sendmail on Unix and does not speak SMTP except on Windows.

See http://zend-framework-community.634137.n4.nabble.com/Zend-Mail-and-Postfix-lt-2-9-td4658920.html for further discussion.

@stewartlord stewartlord Fixed a bug where mail messages were malformed when using the Sendmail
transport on Unix systems with certain versions of sendmail (e.g.
Postfix < 2.9). Specifically, message headers spilled into the body
and the body text was double-spaced.

The source of the problem was the use of CRLF as the EOL sequence
when sending messages via PHP's built-in mail() function on Unix
systems. Despite what the PHP manual says, LF should be used as the
sole EOL character on Unix. This is because mail() is a wrapper for
sendmail on Unix and does not speak SMTP except on Windows.
e3af7c5
Contributor

stewartlord commented Feb 12, 2013

Looks like I didn't cast a wide-enough net. The recipients and subject must also be EOL translated.

@stewartlord stewartlord Cast a wider net for EOL translation. The to and subject headers were
being independantly wrapped. Now we convert line-endings on $to, $subject,
$body and $headers immediately before calling mail().
6dc2cc5

@weierophinney weierophinney added a commit that referenced this pull request Feb 14, 2013

@weierophinney weierophinney Merge branch 'hotfix/3757' into develop
Forward port #3757
f825368

@weierophinney weierophinney added a commit to zendframework/zend-mail that referenced this pull request May 14, 2015

@weierophinney weierophinney Merge branch 'hotfix/3757' 760714d

@weierophinney weierophinney added a commit to zendframework/zend-mail that referenced this pull request May 14, 2015

@weierophinney weierophinney Merge branch 'hotfix/3757' into develop 1fe76d7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment