forks/forgejo
1
0
Fork 0
Code Releases Activity

Do not write HTML in text/plain mail part (#2954)

Browse source 
* Do not write HTML in text/plain mail part

Fixes #2928

* Pass text/plain first, text/html second

* Do not send plain/text email if html2text failed (untested)
This commit is contained in:
Sandro Santilli 2016-05-30 10:18:49 +02:00 committed by Unknwon
parent e9ae926e04
commit d35a1c30f4
1 changed files with 11 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
modules/mailer/mailer.go
View file

@ -18,6 +18,7 @@ import (
"github.com/gogits/gogs/modules/log" "github.com/gogits/gogs/modules/log"
"github.com/gogits/gogs/modules/setting" "github.com/gogits/gogs/modules/setting"
"github.com/jaytaylor/html2text"
) )
type Message struct { type Message struct {
@ -26,14 +27,21 @@ type Message struct {
} }
// NewMessageFrom creates new mail message object with custom From header. // NewMessageFrom creates new mail message object with custom From header.
func NewMessageFrom(to []string, from, subject, body string) *Message { func NewMessageFrom(to []string, from, subject, htmlbody string) *Message {
msg := gomail.NewMessage() msg := gomail.NewMessage()
msg.SetHeader("From", from) msg.SetHeader("From", from)
msg.SetHeader("To", to...) msg.SetHeader("To", to...)
msg.SetHeader("Subject", subject) msg.SetHeader("Subject", subject)
msg.SetDateHeader("Date", time.Now()) msg.SetDateHeader("Date", time.Now())
msg.SetBody("text/plain", body) body, err := html2text.FromString(htmlbody)
msg.AddAlternative("text/html", body) if err != nil {
// TODO: report error ?
msg.SetBody("text/html", htmlbody)
} else {
msg.SetBody("text/plain", body)
// TODO: avoid this (use a configuration switch?)
msg.AddAlternative("text/html", htmlbody)
}
return &Message{ return &Message{
Message: msg, Message: msg,