Skip to content

Email alerter#80

Open
sbzgen wants to merge 1 commit intomichaelhball:mainfrom
sbzgen:email-alerter
Open

Email alerter#80
sbzgen wants to merge 1 commit intomichaelhball:mainfrom
sbzgen:email-alerter

Conversation

@sbzgen
Copy link
Copy Markdown

@sbzgen sbzgen commented Aug 30, 2025

Successor to #3 which needed a rewrite after all the changes in #56.

Untested in your codebase, but this was backported from a hard fork of mine that was heavily rewritten. I have other improvements that can come later but it'd be better to do them one at a time :)

Mail providers can be finicky with what they support and don't support. My solution uses the most compatible approach which most providers will accept.

My solution was to add a command that lets users test their selected alerter to see if it works without any problems, but I don't know if you would want to add that. I could also catch all smtp exceptions and log them.

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

Successfully merging this pull request may close these issues.

1 participant