Browse Source

Merge pull request #11331 from garas/mailtransport-log-subject-3.5

Include Subject and To in email log when using MailTransport
Mark Story 8 years ago
parent
commit
fcb0b36eb4

+ 3 - 0
src/Mailer/Transport/MailTransport.php

@@ -53,6 +53,9 @@ class MailTransport extends AbstractTransport
         $params = isset($this->_config['additionalParameters']) ? $this->_config['additionalParameters'] : null;
         $this->_mail($to, $subject, $message, $headers, $params);
 
+        $headers .= $eol . 'To: ' . $to;
+        $headers .= $eol . 'Subject: ' . $subject;
+
         return ['headers' => $headers, 'message' => $message];
     }
 

+ 4 - 1
tests/TestCase/Mailer/Transport/MailTransportTest.php

@@ -89,6 +89,9 @@ class MailTransportTest extends TestCase
                 '-f'
             );
 
-        $this->MailTransport->send($email);
+        $result = $this->MailTransport->send($email);
+
+        $this->assertContains('Subject: ', $result['headers']);
+        $this->assertContains('To: ', $result['headers']);
     }
 }