Browse Source

HashEmail function should also remove spaces

According to the [Gravatar API](https://en.gravatar.com/site/implement/hash/) whitespace should also be removed from the email, it was not doing this previously.
Andrew Burns 10 years ago
parent
commit
adc1ac689e
1 changed files with 5 additions and 1 deletions
  1. 5 1
      modules/avatar/avatar.go

+ 5 - 1
modules/avatar/avatar.go

@@ -48,8 +48,12 @@ func init() {
 // hash email to md5 string
 // keep this func in order to make this package indenpent
 func HashEmail(email string) string {
+	// https://en.gravatar.com/site/implement/hash/
+	email = strings.TrimSpace(email)
+	email = strings.ToLower(email)
+
 	h := md5.New()
-	h.Write([]byte(strings.ToLower(email)))
+	h.Write([]byte(email))
 	return hex.EncodeToString(h.Sum(nil))
 }