mirror of
https://github.com/healthchecks/healthchecks.git
synced 2026-04-24 22:45:56 +03:00
[PR #685] [MERGED] Add support for SMTP with implicit TLS #1009
Labels
No labels
bug
bug
bug
feature
good-first-issue
new integration
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/healthchecks#1009
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/healthchecks/healthchecks/pull/685
Author: @facorazza
Created: 8/2/2022
Status: ✅ Merged
Merged: 8/9/2022
Merged by: @cuu508
Base:
master← Head:master📝 Commits (4)
d148e7aAdd support for SMTP with implicit TLS3e82366Add SMTP setup example with implict TLS299dc83Restored legacy defaults and added support for EMAIL_USE_SSL4b9f42dMerge branch 'healthchecks:master' into master📊 Changes
4 files changed (+29 additions, -7 deletions)
View changed files
📝
README.md(+20 -7)📝
hc/local_settings.py.example(+1 -0)📝
hc/settings.py(+1 -0)📝
templates/docs/self_hosted_configuration.md(+7 -0)📄 Description
I've added
EMAIL_USE_SSLto the environment variables handled bysettings.pyand changed the default SMTP port to465which is the recommended one over587. I've also set their default value toFalseas they can't be bothTrueat the same time. A user who wants to use either one has to specify the correct one.I've also updated the
README.mdwith the new configuration setup.🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.