-
Notifications
You must be signed in to change notification settings - Fork 61
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
(Nodemailer) Why isn't this supported anymore? #29
Comments
It is not discounted completely, but our new v3 API was a complete change from the old API in form, function, structure, everything. So, it made sense to try something new within our libraries as well. We are open to bringing this back if it's something we absolutely need to bring back, but so far our new library is working really well much fewer dependencies. |
I was looking at the |
That totally makes sense. One thing we were definitely doing in our rewrite was removing the constraints we had from other dependencies but there is definite up side to having the consistent interface. What is your use case? Are you integrating with a 3rd party service that uses nodemailer or is it just that having the ability to switch providers easily is ideal? Something else? Thank you!!! |
@mbernier Initially I wanted to build a general plugin for @Bloggify to send emails. But since I use SendGrid, I ended with creating Still, would be nice to solve this issue one day, so then we can have a general module for sending emails via nodemailer. 😁 |
I updated the title here and marked it as a question. I am interested to see what other people in the Node community think, because this is not the first request on this. We want to support what you all NEED from us, in order to make your jobs easier. It is totally possible for us to support nodemailer in a helper, as well as continue to support our own helpers. And we will do this, if we see that support for this feature is warranted and useful to our community. To everyone else who's reading this - if you want nodemailer support in a helper - throw a thumbs up on this comment or on @IonicaBizau's original comment to let us know!! |
I agree! 👍 |
So... what is the recommended library for sending e-mails in nodemailer using SendGrid? This is not very clear to me. |
@elboletaire We don't have a recommended nodemailer library currently. We only officially support this Node.js library: https://github.com/sendgrid/sendgrid-nodejs I hope that helps. Thanks! |
I also would like to see nodemailer continue to be supported. Not that I have any problem with the official node module, it's just that I think it's better to have a provider agnostic way to send email. If this module uses the official sendgrid module under the hood and the official module does all the heavy lifting, isn't it only a matter of keeping the underlying module up to date and keeping the nodemailer api working? With all that said, I understand if you don't want to put more resources into this module, in that case, maybe it's best if you let the community pick it up. I already see pull requests like this one that should move this forward. |
I'm choosing an emailer solution right now for an MVP project I am working on and in the past I have always chosen SendGrid as, in the past, it's been trivial to wire it into NodeMailer. I like NodeMailer because, during development or while running integration tests, I can easily drop back to the nodemailer-stub-transport or my mac's built-in Sendmail smtp server. I've looked at the 'official' sendgrid-nodejs library and the huge number of outstanding issues / PRs are a concern, and there does not appear to be any equivalent to the So I second the comment from @mderazon above. If Sendgrid does not have the resources to support its nodemailer transport then please could you release it into the wild and let the broader community maintain it. In the meantime I'll use this version which appears to be the most current fork. |
Got set up with SendGrid to provide SMTP service for sending actual emails. Right now it's just sending emails to KeystoneJS admins when there's a new enquiry. The emails need styled, and they're set up in an over-complicated manner. They need to be simplified and given NGC branding. We'll eventually re-use this for sending emails to users, etc. Nodemailer's integration with SendGrid is a bit sketchy right now: we're relying on a dumb fork of the no-longer-officially supported library. See this issue: sendgrid/nodemailer-sendgrid-transport#29 The actual SendGrid account is managed through Azure.
Hi @mbernier, The reason I came to this thread was because I was going to setup a Ghost blog with Sendgrid and Ghost uses nodemailer, but I don't want to add username and password to my account in the blog config, in my opinion that is just wrong. I want to use an API key. I don't know how many other projects use nodemailer, but if there are many other big projects like Ghost that does then maybe it would be good to either make pull requests to said projects to use your new official API libraries (that looks pretty nice btw), or to make a basic nodemailer wrapper around your new client so that it supports the basics of sending emails using a secure API key. Cheers, Robert |
I have just tried |
Hi @moravcik, Yes, the current version (that supports the v2 Web API), while no longer supported, is not being sunset. Welcome to SendGrid! Glad to have you aboard :) With Best Regards, Elmer |
Looking forward for an official SendGrid V3 transport for |
Eh, I don't like their native API very much. Lots of object initialization. Doesn't feel very idiomatic for Node.js. I just decided to call their API directly, and the code came out much cleaner: const api = require('law-axios')
const sendgrid = api({
json: true,
domain: 'https://api.sendgrid.com/v3/',
headers: {
Authorization: 'Bearer ' + process.env.SENDGRID_KEY
}
})
const body = {
personalizations: [{to: [{email: to, name: toName}], subject}],
from: {
email: from,
name: fromName,
},
reply_to: {
email: from,
name: fromName,
},
subject,
content: [
{type: 'text/html', value: html}
]
}
sendgrid('POST /mail/send', body, done) Yeah, the cross-compatibility would be nice, but I'm willing to swap out 30 lines of code in the event that we switch carriers. |
Thanks for sharing your solution @bitmage! If you have a moment, we would like to send you some swag. Have you checked out the new version of our SDK? It should be released very soon. |
I'm on a project where we use SES and we also need to send out email. Nodemailer is really easy to use with ses, smtp, and a variety of communication transports. 👎 for dropping nodemailer support. |
Hi @jsdevel, Please follow this issue for progress :) |
That's interesting @oliviertassinari, I was not aware of this feature. I will look into whether that would be appropriate for this repo. For now, I'm thinking the announcement on the README should be sufficient. With Best Regards, Elmer |
Was there any update regarding the integration of the new API from sendgrid? |
Yeah, switch to latest version of nodemailer and you have it: https://github.com/nodemailer/nodemailer-sendgrid |
I think it would be great to link to the Following the the installation instructions in this repo: https://github.com/sendgrid/nodemailer-sendgrid-transport/blob/master/USAGE.md
The For anyone reading this, I would recommend using the |
Thanks for the heads up @DaddyWarbucks! I've added this to our backlog for review, it seems like a solid alternative until we can get this repo re-created here. |
Can I use smtp transport of nodemailer with sendgrid? What's correct configuration? |
Hello @sheerun, We have a solution in the works, but it's still on the backlog. With Best Regards, Elmer |
If this is no longer supported, you should take down https://sendgrid.com/blog/sending-email-nodemailer-sendgrid/ It's the top link when searching "sendgrid nodemailer" For future readers: this is the latest solution https://github.com/nodemailer/nodemailer-sendgrid |
As most people should have known by now, this project was moved to the main repo under the |
ematiu created a PR #71 to update the README.md. It would be great to see the verbiage reviewed and get that PR merged in. That way others who came across this repo via the SendGrid blog article @kusold mentioned, can be made aware that this is replaced by the npm package |
Maybe can I adopt it? Or is there a specific reason for deprecation?
The text was updated successfully, but these errors were encountered: