From: Andrew Burns <andrew@wyominghealthfairs.com>
Date: Wed, 3 Dec 2014 23:19:35 +0000 (-0700)
Subject: Use the avatar.HashEmail function instead of hashing email directly.  Fixes #700
X-Git-Tag: v0.9.99~1605^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=00cf3e4dabf810751bab3c83787c343f5ac6a88c;p=gitea.git

Use the avatar.HashEmail function instead of hashing email directly.  Fixes #700
---

diff --git a/models/user.go b/models/user.go
index 5c6101e62c..a83051dbbd 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
 }