search

hacklabr / paralapraca

GNU Affero General Public License v3.0
0 stars 4 forks source link

[Signup] Campo email não deve permitir caracteres inválidos #387

Closed laurybueno closed 6 years ago

laurybueno commented 7 years ago

"ç" é um exemplo de letra aceita incorretamente

virgilio commented 7 years ago

Já tem uma validação nativa do navegador, colocar uma via JS?

laurybueno commented 7 years ago

acho que o campo nem é to type email ainda. Só isso, já resolve bem imagino.

virgilio commented 7 years ago

No signup (em /accounts/signup ) é, no login, não (até porque é email/username)... Uma pergunta, qual página dá acesso ao signup? eu só consigo entrar pelo link direto da url de accounts, não achei nenhum link que chega nela.

laurybueno commented 7 years ago

o type do campo é email e ele permite caracteres como "ç"? Essa é nova pra mim. Talvez, JS seja o caminho então (validação no django também é aceitável)

Nós não temos um signup aberto mesmo. O cliente manda a URL diretamente para possíveis interessados.

leopiccionia commented 7 years ago

Aparentemente, o que o Django estava fazendo era converter UTF8 em Punycode. Alguns provedores de email permitem caracteres UTF8 tanto antes quanto depois da arroba (há RFCs nesse sentido), precisa ver se suportamos isso.

virgilio commented 7 years ago

Opa, isso aí é ainda outra questão, olha só: https://i.imgur.com/iFCIIL6.png

Aqui ele não deixa dar submit

laurybueno commented 7 years ago

parece que antes do @ ele não deixa e depois deixa

laurybueno commented 7 years ago

@virgilio dá uma olhada no usuário laurylaury na produção. Ele tem o ç depois do @. Vê em que telas isso é um problema (acho que é na edição de perfil)

laurybueno commented 7 years ago

@virgilio, era para ter um "ç" nesse email. Ele não aparece. Essa é a tela de edição de perfil do laurylaury

screenshot from 2017-10-19 15-34-06

virgilio commented 7 years ago

Eu testei localmente o signup e o edit e funcionou. Mas precisaríamos achar um servidor que tenha caracter especial pra ver como se comporta o envio de e-mail, exceto por isso, não parece haver problemas naquele form. (eu mandei outra mensagem e apaguei sem querer, vcs devem ter recebido por email)

virgilio commented 7 years ago

Testarei

virgilio commented 7 years ago

Agora ficou bem isolada, no admin tá correto, então, é só na tela mesmo, valeu!

virgilio commented 7 years ago

@laurybueno eu não estou conseguindo reproduzir em http://ava.paralapraca.org.br, config_account_ava.png

virgilio commented 7 years ago

No localhost a mesma coisa, ç entra no sistema normal e é apresentada normalmente

laurybueno commented 6 years ago

Deve estar correto já então.