Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SMTP configuration not being passed to nodemailer #29

Closed
rbaprado opened this issue Oct 27, 2014 · 7 comments
Closed

SMTP configuration not being passed to nodemailer #29

rbaprado opened this issue Oct 27, 2014 · 7 comments

Comments

@rbaprado
Copy link

It seems that the emailSettings are not being passed on to the smtpTransport, since I specified them and digging in to Email.prototype.send, line 58, the emailSettings are not being passed on and, due to that, I got an ECONNREFUSED:

// send email with nodemailer
var transporter = nodemailer.createTransport(that.transport(options)); // options are not including my emailSettings
transporter.sendMail(options, function(err, res){
  if(err) return done(err);
  transporter.close(); // shut down the connection pool, no more messages
  done(null, res);
});
@clong365
Copy link

+1

@jaaxxm
Copy link

jaaxxm commented Jan 11, 2015

i've fixed this issue by passing that.config.emailSettings to nodemailer.createTransport method.

var transporter = nodemailer.createTransport(that.config.emailSettings); 

@jfgirard
Copy link

jfgirard commented Feb 6, 2015

+1

@shawnbissell
Copy link

Too bad @zemirco has abandoned this project :( About 8 different people have fixed this same problem since Sept 2014 when it was introduced, but the pull requests have not yet been merged back. I'd suggest switching to @killerbobjr repo for lockit. He has this fix and his package.json files are corrected.

Just change your package.json
"lockit": "killerbobjr/lockit",

@valnub
Copy link

valnub commented Jun 28, 2015

Same problem here

@zemirco
Copy link
Owner

zemirco commented Jun 30, 2015

Issue is fixed now - https://github.com/zemirco/lockit-sendmail/blob/master/index.js#L59.

I'm currently working on cleaning and updating all lockit modules. New release will be out soon.

@valnub
Copy link

valnub commented Jun 30, 2015

Cool, sanks tu ze Mirco :)

zemirco added a commit that referenced this issue Jul 6, 2015
@zemirco zemirco closed this as completed Jul 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants