summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--utils/config.go5
-rw-r--r--utils/mail.go98
2 files changed, 71 insertions, 32 deletions
diff --git a/utils/config.go b/utils/config.go
index 745887c70..418041706 100644
--- a/utils/config.go
+++ b/utils/config.go
@@ -222,6 +222,11 @@ func LoadConfig(fileName string) {
config.ServiceSettings.Domain = os.Getenv("MATTERMOST_DOMAIN")
}
+ // Validates our mail settings
+ if err := CheckMailSettings(); err != nil {
+ l4g.Error("Email settings are not valid err=%v", err)
+ }
+
configureLog(config.LogSettings)
Cfg = &config
diff --git a/utils/mail.go b/utils/mail.go
index bf1cc9d46..2fb7f801d 100644
--- a/utils/mail.go
+++ b/utils/mail.go
@@ -13,32 +13,27 @@ import (
"net/smtp"
)
-func SendMail(to, subject, body string) *model.AppError {
-
- fromMail := mail.Address{"", Cfg.EmailSettings.FeedbackEmail}
- toMail := mail.Address{"", to}
-
- headers := make(map[string]string)
- headers["From"] = fromMail.String()
- headers["To"] = toMail.String()
- headers["Subject"] = subject
- headers["MIME-version"] = "1.0"
- headers["Content-Type"] = "text/html"
-
- message := ""
- for k, v := range headers {
- message += fmt.Sprintf("%s: %s\r\n", k, v)
- }
- message += "\r\n<html><body>" + body + "</body></html>"
-
+func CheckMailSettings() *model.AppError {
if len(Cfg.EmailSettings.SMTPServer) == 0 {
- l4g.Warn("Skipping sending of email because EmailSettings are not configured")
- return nil
+ return model.NewAppError("CheckMailSettings", "No email settings present, mail will not be sent", "")
}
+ conn, err := connectToSMTPServer()
+ if err != nil {
+ return err
+ }
+ defer conn.Close()
+ c, err2 := newSMTPClient(conn)
+ if err2 != nil {
+ return err
+ }
+ defer c.Quit()
+ defer c.Close()
- host, _, _ := net.SplitHostPort(Cfg.EmailSettings.SMTPServer)
+ return nil
+}
- auth := smtp.PlainAuth("", Cfg.EmailSettings.SMTPUsername, Cfg.EmailSettings.SMTPPassword, host)
+func connectToSMTPServer() (net.Conn, *model.AppError) {
+ host, _, _ := net.SplitHostPort(Cfg.EmailSettings.SMTPServer)
var conn net.Conn
var err error
@@ -51,38 +46,77 @@ func SendMail(to, subject, body string) *model.AppError {
conn, err = tls.Dial("tcp", Cfg.EmailSettings.SMTPServer, tlsconfig)
if err != nil {
- return model.NewAppError("SendMail", "Failed to open TLS connection", err.Error())
+ return nil, model.NewAppError("SendMail", "Failed to open TLS connection", err.Error())
}
} else {
conn, err = net.Dial("tcp", Cfg.EmailSettings.SMTPServer)
if err != nil {
- return model.NewAppError("SendMail", "Failed to open connection", err.Error())
+ return nil, model.NewAppError("SendMail", "Failed to open connection", err.Error())
}
}
- defer conn.Close()
+ return conn, nil
+}
+func newSMTPClient(conn net.Conn) (*smtp.Client, *model.AppError) {
+ host, _, _ := net.SplitHostPort(Cfg.EmailSettings.SMTPServer)
c, err := smtp.NewClient(conn, host)
if err != nil {
l4g.Error("Failed to open a connection to SMTP server %v", err)
- return model.NewAppError("SendMail", "Failed to open TLS connection", err.Error())
+ return nil, model.NewAppError("SendMail", "Failed to open TLS connection", err.Error())
}
- defer c.Quit()
- defer c.Close()
-
// GO does not support plain auth over a non encrypted connection.
// so if not tls then no auth
+ auth := smtp.PlainAuth("", Cfg.EmailSettings.SMTPUsername, Cfg.EmailSettings.SMTPPassword, host)
if Cfg.EmailSettings.UseTLS {
if err = c.Auth(auth); err != nil {
- return model.NewAppError("SendMail", "Failed to authenticate on SMTP server", err.Error())
+ return nil, model.NewAppError("SendMail", "Failed to authenticate on SMTP server", err.Error())
}
}
+ return c, nil
+}
+
+func SendMail(to, subject, body string) *model.AppError {
+
+ fromMail := mail.Address{"", Cfg.EmailSettings.FeedbackEmail}
+ toMail := mail.Address{"", to}
+
+ headers := make(map[string]string)
+ headers["From"] = fromMail.String()
+ headers["To"] = toMail.String()
+ headers["Subject"] = subject
+ headers["MIME-version"] = "1.0"
+ headers["Content-Type"] = "text/html"
+
+ message := ""
+ for k, v := range headers {
+ message += fmt.Sprintf("%s: %s\r\n", k, v)
+ }
+ message += "\r\n<html><body>" + body + "</body></html>"
+
+ if len(Cfg.EmailSettings.SMTPServer) == 0 {
+ l4g.Warn("Skipping sending of email because EmailSettings are not configured")
+ return nil
+ }
+
+ conn, err1 := connectToSMTPServer()
+ if err1 != nil {
+ return err1
+ }
+ defer conn.Close()
+
+ c, err2 := newSMTPClient(conn)
+ if err2 != nil {
+ return err2
+ }
+ defer c.Quit()
+ defer c.Close()
- if err = c.Mail(fromMail.Address); err != nil {
+ if err := c.Mail(fromMail.Address); err != nil {
return model.NewAppError("SendMail", "Failed to add from email address", err.Error())
}
- if err = c.Rcpt(toMail.Address); err != nil {
+ if err := c.Rcpt(toMail.Address); err != nil {
return model.NewAppError("SendMail", "Failed to add to email address", err.Error())
}