You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
What is broken is that the auth options are not passed as options to the createTransport method anymore, but instead are set on the NodeMailer instance returned by that method and NodeMailer couldn't care any less about such things. It is the SMTPTransport that checks the auth options here: https://github.com/nodemailer/nodemailer-smtp-transport/blob/v1.1.0/src/smtp-transport.js#L145
So, this pr should probably be reversed and a new release be made asap as it likely breaks every auth using email sending node-red out there.
The text was updated successfully, but these errors were encountered:
timolehto
added a commit
to timolehto/node-red-nodes
that referenced
this issue
Feb 21, 2017
apologies - I'm on a slow 2g link and didn't see this - must have crossed in the ether.. yes will probably accept yours later when back on a decent link as it's a better fix
Broken by this pr: #277
What is broken is that the auth options are not passed as options to the createTransport method anymore, but instead are set on the NodeMailer instance returned by that method and NodeMailer couldn't care any less about such things. It is the SMTPTransport that checks the auth options here:
https://github.com/nodemailer/nodemailer-smtp-transport/blob/v1.1.0/src/smtp-transport.js#L145
So, this pr should probably be reversed and a new release be made asap as it likely breaks every auth using email sending node-red out there.
The text was updated successfully, but these errors were encountered: