Merge branch 'hotfix/3757'

Close zendframework#3757
dineshkummarc · Feb 14, 2013 · 2a51b16 · 2a51b16
2 parents 44b7108 + 0c581ac
commit 2a51b16
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 6 deletions.
diff --git a/library/Zend/Mail/Transport/Sendmail.php b/library/Zend/Mail/Transport/Sendmail.php
@@ -127,6 +127,15 @@ public function send(Mail\Message $message)
         $headers = $this->prepareHeaders($message);
         $params  = $this->prepareParameters($message);
 
+        // On *nix platforms, we need to replace \r\n with \n
+        // sendmail is not an SMTP server, it is a unix command - it expects LF
+        if (!$this->isWindowsOs()) {
+            $to      = str_replace("\r\n", "\n", $to);
+            $subject = str_replace("\r\n", "\n", $subject);
+            $body    = str_replace("\r\n", "\n", $body);
+            $headers = str_replace("\r\n", "\n", $headers);
+        }
+
         call_user_func($this->callable, $to, $subject, $body, $headers, $params);
     }
 

diff --git a/tests/ZendTest/Mail/Transport/SendmailTest.php b/tests/ZendTest/Mail/Transport/SendmailTest.php
@@ -83,12 +83,12 @@ public function testReceivesMailArtifactsOnUnixSystems()
         $this->assertEquals('ZF DevTeam <[email protected]>', $this->to);
         $this->assertEquals('Testing Zend\Mail\Transport\Sendmail', $this->subject);
         $this->assertEquals('This is only a test.', trim($this->message));
-        $this->assertNotContains("To: ZF DevTeam <[email protected]>\r\n", $this->additional_headers);
-        $this->assertContains("Cc: [email protected]\r\n", $this->additional_headers);
-        $this->assertContains("Bcc: \"CR-Team, ZF Project\" <[email protected]>\r\n", $this->additional_headers);
-        $this->assertContains("From: [email protected],\r\n Matthew <[email protected]>\r\n", $this->additional_headers);
-        $this->assertContains("X-Foo-Bar: Matthew\r\n", $this->additional_headers);
-        $this->assertContains("Sender: Ralph Schindler <[email protected]>\r\n", $this->additional_headers);
+        $this->assertNotContains("To: ZF DevTeam <[email protected]>\n", $this->additional_headers);
+        $this->assertContains("Cc: [email protected]\n", $this->additional_headers);
+        $this->assertContains("Bcc: \"CR-Team, ZF Project\" <[email protected]>\n", $this->additional_headers);
+        $this->assertContains("From: [email protected],\n Matthew <[email protected]>\n", $this->additional_headers);
+        $this->assertContains("X-Foo-Bar: Matthew\n", $this->additional_headers);
+        $this->assertContains("Sender: Ralph Schindler <[email protected]>\n", $this->additional_headers);
         $this->assertEquals('-R hdrs -f [email protected]', $this->additional_parameters);
     }