From 99b8eef7e373a000f35016415ba2148f939c1155 Mon Sep 17 00:00:00 2001 From: Thomas Balthazar Date: Wed, 18 May 2016 13:54:33 +0200 Subject: Send email notification when username changed (#3022) --- api/user.go | 44 +++++++++++++++++++++++++++------- i18n/en.json | 16 +++++++++++++ i18n/fr.json | 16 +++++++++++++ templates/username_change_body.html | 41 +++++++++++++++++++++++++++++++ templates/username_change_subject.html | 1 + 5 files changed, 109 insertions(+), 9 deletions(-) create mode 100644 templates/username_change_body.html create mode 100644 templates/username_change_subject.html diff --git a/api/user.go b/api/user.go index 559cda076..da94a0ff5 100644 --- a/api/user.go +++ b/api/user.go @@ -6,15 +6,6 @@ package api import ( "bytes" "fmt" - l4g "github.com/alecthomas/log4go" - "github.com/disintegration/imaging" - "github.com/golang/freetype" - "github.com/gorilla/mux" - "github.com/mattermost/platform/einterfaces" - "github.com/mattermost/platform/model" - "github.com/mattermost/platform/store" - "github.com/mattermost/platform/utils" - "github.com/mssola/user_agent" "hash/fnv" "html/template" "image" @@ -30,6 +21,16 @@ import ( "strconv" "strings" "time" + + l4g "github.com/alecthomas/log4go" + "github.com/disintegration/imaging" + "github.com/golang/freetype" + "github.com/gorilla/mux" + "github.com/mattermost/platform/einterfaces" + "github.com/mattermost/platform/model" + "github.com/mattermost/platform/store" + "github.com/mattermost/platform/utils" + "github.com/mssola/user_agent" ) func InitUser() { @@ -1291,6 +1292,10 @@ func updateUser(c *Context, w http.ResponseWriter, r *http.Request) { } } + if rusers[0].Username != rusers[1].Username { + sendEmailChangeUsernameAndForget(c, rusers[1].Username, rusers[0].Username, rusers[0].Email, c.GetSiteURL()) + } + rusers[0].Password = "" rusers[0].AuthData = new(string) *rusers[0].AuthData = "" @@ -1832,6 +1837,27 @@ func SendEmailChangeVerifyEmailAndForget(c *Context, userId, newUserEmail, siteU }() } +func sendEmailChangeUsernameAndForget(c *Context, oldUsername, newUsername, email, siteURL string) { + go func() { + + subjectPage := utils.NewHTMLTemplate("username_change_subject", c.Locale) + subjectPage.Props["Subject"] = c.T("api.templates.username_change_subject", + map[string]interface{}{"TeamDisplayName": utils.Cfg.TeamSettings.SiteName}) + subjectPage.Props["SiteName"] = utils.Cfg.TeamSettings.SiteName + + bodyPage := utils.NewHTMLTemplate("email_change_body", c.Locale) + bodyPage.Props["SiteURL"] = siteURL + bodyPage.Props["Title"] = c.T("api.templates.username_change_body.title") + bodyPage.Html["Info"] = template.HTML(c.T("api.templates.username_change_body.info", + map[string]interface{}{"TeamDisplayName": utils.Cfg.TeamSettings.SiteName, "NewUsername": newUsername})) + + if err := utils.SendMail(email, subjectPage.Render(), bodyPage.Render()); err != nil { + l4g.Error(utils.T("api.user.send_email_change_username_and_forget.error"), err) + } + + }() +} + func updateUserNotify(c *Context, w http.ResponseWriter, r *http.Request) { props := model.MapFromJson(r.Body) diff --git a/i18n/en.json b/i18n/en.json index 6ecfece69..042f6d65c 100644 --- a/i18n/en.json +++ b/i18n/en.json @@ -1203,6 +1203,18 @@ "id": "api.templates.email_change_subject", "translation": "Your email address has changed for {{.TeamDisplayName}}" }, + { + "id": "api.templates.username_change_body.info", + "translation": "Your username for {{.TeamDisplayName}} has been changed to {{.NewUsername}}.
If you did not make this change, please contact the system administrator." + }, + { + "id": "api.templates.username_change_body.title", + "translation": "You updated your username" + }, + { + "id": "api.templates.username_change_subject", + "translation": "Your username has changed for {{.TeamDisplayName}}" + }, { "id": "api.templates.email_change_verify_body.button", "translation": "Verify Email" @@ -1639,6 +1651,10 @@ "id": "api.user.send_email_change_email_and_forget.error", "translation": "Failed to send email change notification email successfully err=%v" }, + { + "id": "api.user.send_email_change_username_and_forget.error", + "translation": "Failed to send username change notification email successfully err=%v" + }, { "id": "api.user.send_email_change_verify_email_and_forget.error", "translation": "Failed to send email change verification email successfully err=%v" diff --git a/i18n/fr.json b/i18n/fr.json index b5661b668..f88fbc522 100644 --- a/i18n/fr.json +++ b/i18n/fr.json @@ -1103,6 +1103,18 @@ "id": "api.templates.email_change_subject", "translation": "Votre adresse électronique a été modifiée pour {{.TeamDisplayName}}" }, + { + "id": "api.templates.username_change_body.info", + "translation": "Votre nom d'utilisateur pour {{.TeamDisplayName}} a été modifiée en {{.NewUsername}}.
Si vous n'êtes pas à l'origine de cette action, veuillez contacter votre administrateur système." + }, + { + "id": "api.templates.username_change_body.title", + "translation": "Vous avez mis à jour votre nom d'utilisateur." + }, + { + "id": "api.templates.username_change_subject", + "translation": "Votre nom d'utilisateur a été modifiée pour {{.TeamDisplayName}}" + }, { "id": "api.templates.email_change_verify_body.button", "translation": "Vérifier votre adresse électronique" @@ -1495,6 +1507,10 @@ "id": "api.user.send_email_change_email_and_forget.error", "translation": "Impossible d'envoyer le courriel de changement d'adresse électronique err=%v" }, + { + "id": "api.user.send_email_change_username_and_forget.error", + "translation": "Impossible d'envoyer le courriel de changement de nom d'utilisateur err=%v" + }, { "id": "api.user.send_email_change_verify_email_and_forget.error", "translation": "Impossible d'envoyer le courriel de changement d'adresse électronique err=%v" diff --git a/templates/username_change_body.html b/templates/username_change_body.html new file mode 100644 index 000000000..56c383d49 --- /dev/null +++ b/templates/username_change_body.html @@ -0,0 +1,41 @@ +{{define "username_change_body"}} + + + + + +
+ + + + +
+ + + + + + + + + {{template "email_footer" . }} + +
+ +
+ + + + + + {{template "email_info" . }} + +
+

{{.Props.Title}}

+

{{.Html.Info}}

+
+
+
+
+ +{{end}} diff --git a/templates/username_change_subject.html b/templates/username_change_subject.html new file mode 100644 index 000000000..1c94bb7c0 --- /dev/null +++ b/templates/username_change_subject.html @@ -0,0 +1 @@ +{{define "username_change_subject"}}[{{.Props.SiteName}}] {{.Props.Subject}}{{end}} -- cgit v1.2.3-1-g7c22