From 00cf3e4dabf810751bab3c83787c343f5ac6a88c Mon Sep 17 00:00:00 2001 From: Andrew Burns Date: Wed, 3 Dec 2014 16:19:35 -0700 Subject: [PATCH] Use the avatar.HashEmail function instead of hashing email directly. Fixes #700 --- models/user.go | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/models/user.go b/models/user.go index 5c6101e62..a83051dbb 100644 --- a/models/user.go +++ b/models/user.go @@ -21,6 +21,7 @@ import ( "github.com/Unknwon/com" "github.com/nfnt/resize" + "github.com/gogits/gogs/modules/avatar" "github.com/gogits/gogs/modules/base" "github.com/gogits/gogs/modules/git" "github.com/gogits/gogs/modules/log" @@ -276,8 +277,8 @@ func CreateUser(u *User) error { } u.LowerName = strings.ToLower(u.Name) - u.Avatar = base.EncodeMd5(u.Email) u.AvatarEmail = u.Email + u.Avatar = avatar.HashEmail(u.AvatarEmail) u.Rands = GetUserSalt() u.Salt = GetUserSalt() u.EncodePasswd() @@ -436,6 +437,11 @@ func UpdateUser(u *User) error { u.Description = u.Description[:255] } + if u.AvatarEmail == "" { + u.AvatarEmail = u.Email + } + u.Avatar = avatar.HashEmail(u.AvatarEmail) + _, err = x.Id(u.Id).AllCols().Update(u) return err }