Browse Source

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

Andrew Burns 10 years ago
parent
commit
00cf3e4dab
1 changed files with 7 additions and 1 deletions
  1. 7 1
      models/user.go

+ 7 - 1
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
 }