Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Gravatar hash is computed from the un-lowercased email, so mixed-case accounts render an identicon

Abierto Apto para principiantes
#1,604 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
85/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
go, typescript
Área
backend

Línea de trabajo

Comienza en pkg/gravatar/gravatar.go, en GetAvatarURL, y compara el tratamiento de su entrada con la conversión a minúsculas y el recorte que ya se utilizan en ui/src/pages/Users/Settings/Profile/index.tsx. Verifica que la URL generada use el correo electrónico normalizado para que las direcciones almacenadas con mayúsculas y minúsculas mezcladas resuelvan el Gravatar registrado en lugar del fallback de identicon.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Describe the bug

pkg/gravatar.GetAvatarURL hashes the email address without lowercasing it:

// pkg/gravatar/gravatar.go
func GetAvatarURL(baseURL, email string) string {
	hasher := sha256.Sum256([]byte(strings.TrimSpace(email)))
	hash := hex.EncodeToString(hasher[:])
	return baseURL + hash
}

The Gravatar specification requires the address to be trimmed and lowercased
before hashing, so any account whose stored address contains an uppercase letter
hashes to an address Gravatar does not know. The user's avatar is never found and
the identicon fallback (&d=identicon, added by the UI) is rendered instead.

Two details make this hard to work around from outside:

  1. selectedAvatar in internal/service/siteinfo_common/siteinfo_service.go
    recomputes the Gravatar URL from the stored address on every response, for
    both default_avatar: gravatar and an explicit per-user avatar.type: gravatar. Selecting "Gravatar" in Settings → Profile stores the correct URL
    (the UI hashes correctly, see below) but the server overwrites it on read.
  2. Nothing in the backend normalises a stored address. grep -rn ToLower internal/ | grep -i mail returns nothing, and the external-login path copies
    the provider's address verbatim
    (internal/service/user_external_login/user_external_login_service.go:258,
    userInfo.EMail = externalUserInfo.Email). With an OIDC/OAuth2 connector the
    casing is whatever the identity provider sends, so accounts are created
    mis-cased without any user or admin action.

The web UI already does this correctly, which is why the bug is easy to miss —
ui/src/pages/Users/Settings/Profile/index.tsx:

const str = res.e_mail.toLowerCase().trim();
const hash = sha256(str);

So the avatar preview in Settings → Profile shows the user's real Gravatar while
every other surface (question lists, answers, comments, user cards) shows an
identicon. Frontend and backend disagree on the hash for the same account.

To Reproduce

  1. Admin → Settings → Users: set "Default avatar" to Gravatar.

  2. Create a user whose email address contains an uppercase letter, e.g.
    Example.User@example.com, and register that lowercase address at
    gravatar.com with an avatar image.

  3. Open any page listing that user (or their profile).

  4. An identicon is rendered instead of the Gravatar avatar.

  5. Confirm the cause without Answer:

    $ printf '%s' 'example.user@example.com' | shasum -a 256   # what Gravatar expects
    $ printf '%s' 'Example.User@example.com' | shasum -a 256   # what Answer sends
    

    Requesting https://www.gravatar.com/avatar/<hash>?d=404 returns 200 for the
    first hash and 404 for the second.

  6. Settings → Profile shows the correct avatar in its preview, because the UI
    lowercases before hashing.

Expected behavior

GetAvatarURL lowercases the address before hashing, matching the Gravatar
specification and the existing frontend behaviour, so a registered Gravatar is
found regardless of the casing of the stored address.

Screenshots

n/a — the hash comparison in step 5 shows the failure directly.

Platform

  • Device: Desktop
  • OS: Linux (container), macOS client
  • Browser and version: Chrome (any)
  • Version: v2.0.2
Lenguaje dominante
Go
Estrellas
15.7k
Forks
1.4k
Merge medio
1 d 20 h
PR fusionados (30 d)
6

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de apache/answer

Todos los issues de apache/answer

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.