Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Use envelope options for sendmail transport #111

Merged
merged 1 commit into from

2 participants

@txase

Previously, the envelope information was unused. Sendmail would
construct the envelope itself from the message headers.

Chase Douglas Use envelope information for sendmail transport
Previously, the envelope information was unused. Sendmail would
construct the envelope itself from the message headers.
b14bf2a
@andris9 andris9 merged commit 7ae622b into from
@andris9
Owner

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 17, 2012
  1. Use envelope information for sendmail transport

    Chase Douglas authored
    Previously, the envelope information was unused. Sendmail would
    construct the envelope itself from the message headers.
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 4 deletions.
  1. +2 −4 lib/engines/sendmail.js
View
6 lib/engines/sendmail.js
@@ -34,10 +34,8 @@ function SendmailTransport(config){
*/
SendmailTransport.prototype.sendMail = function(emailMessage, callback) {
- // sendmail strips this header line by itself
- emailMessage.options.keepBcc = true;
-
- var sendmail = spawn(this.path, ["-t"].concat(this.args));
+ var envelope = emailMessage.getEnvelope();
+ var sendmail = spawn(this.path, ["-f"].concat(envelope.from).concat(envelope.to));
sendmail.on('exit', function (code) {
var msg = "Sendmail exited with "+code;
Something went wrong with that request. Please try again.