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

Error handler for bad transport #44

Open
dkyeremeh opened this issue Sep 26, 2024 · 0 comments
Open

Error handler for bad transport #44

dkyeremeh opened this issue Sep 26, 2024 · 0 comments

Comments

@dkyeremeh
Copy link

dkyeremeh commented Sep 26, 2024

First of all, I would like to express my appreciation for this project; especially for the ability to use multiple data stores. I have been able to use it with redis successfully. Thanks!!

I have a suggestion:

I have had a scenario where one or more transports were misconfigured (typo in smtp credential). But there were no errors because the other transports were working just fine, and the emails got sent.

We are currently able to pass an error handler through opts.onError. But this gets called only when the sending of email fails, and all attempts are exhausted. As long as a functional transport exists, opts.onError will never get called; and we can't identify the misconfiguration.

It would be great if we could specify some form of configuration/error handler that will get called when a particular transport consistently fails. This will allow us to catch misconfigurations, and provide feedback if the smtp server is no longer available (for whatever reason).

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

1 participant