turicas/brasil.io

Corrigir login por usuário/email

Open

#511 opened on Dec 8, 2020

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Python (150 forks)github user discovery
help wanted

Repository metrics

Stars
 (1,032 stars)
PR merge metrics
 (PR metrics pending)

Description

Recebo muitos emails de usuários que não conseguem logar, mesmo depois de trocar a senha. O motivo é que preenchem o email no campo "username" e, com isso, o login falha. Para resolver esse problema, criei um backend de autenticação que usa email se detectar uma @ e username caso contrário, o problema é que muitos usuários possuem @ no username: alguns colocaram o próprio email no username, já outros colocaram @fulano. É necessário:

  • Trocar o validador do cadastro para não permitir @ no username (provavelmente teremos que mudar o model User e sobrescrever username para passar o validador modificado)
  • Indicar melhor no cadastro e no login (talvez com uma @ antes do campo username) a diferença entre email/login (e talvez indicar no login "username ou email")
  • Migrar (de maneira automatizada ou não) os usernames que possuem @ para usernames que não possuem

Issues relacionadas no Sentry (para quem tem acesso):

Contributor guide

Research direction
Review the custom auth backend and modify the User model's username validator to reject '@', then update registration and login forms to clarify username vs email, and create a migration to clean up existing usernames containing '@'.
Tech stack
python
Domain
backend
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitPythonDjango
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60