Merge pull request #44 from nliautaud/41-from-server-noreply

nliautaud · web-flow · commit 4167d061c789 · 2019-01-27T13:58:25.000+01:00
fix #41 use no-reply@server as from header
diff --git a/p01-contact/src/P01contact_Form.php b/p01-contact/src/P01contact_Form.php
@@ -427,9 +427,8 @@ public function sendMail()
     private function mailHeaders($name, $email, $mime_boundary)
     {
         $encoded_name = $this->encodeHeader($name);
-        $headers  = "From: $encoded_name";
+        $headers  = "From: $encoded_name <no-reply@" . SERVERNAME . ">\n";
         if ($email) {
-            $headers .= " <$email>\n";
             $headers .= "Reply-To: $encoded_name <$email>\n";
             $headers .= "Return-Path: $encoded_name <$email>";
         }

Original file line number	Diff line number	Diff line change
`@@ -427,9 +427,8 @@ public function sendMail()`
`427`	`427`	`private function mailHeaders($name, $email, $mime_boundary)`
`428`	`428`	`{`
`429`	`429`	`$encoded_name = $this->encodeHeader($name);`
`430`		`- $headers = "From: $encoded_name";`
	`430`	`+ $headers = "From: $encoded_name <no-reply@" . SERVERNAME . ">\n";`
`431`	`431`	`if ($email) {`
`432`		`- $headers .= " <$email>\n";`
`433`	`432`	`$headers .= "Reply-To: $encoded_name <$email>\n";`
`434`	`433`	`$headers .= "Return-Path: $encoded_name <$email>";`
`435`	`434`	`}`