Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Desabilitar Senha Complexa #4

Open
uira opened this issue Jun 15, 2017 · 2 comments
Open

Desabilitar Senha Complexa #4

uira opened this issue Jun 15, 2017 · 2 comments

Comments

@uira
Copy link
Member

uira commented Jun 15, 2017

A senha não precisa ser com letras números e acentos.

@uira uira added the bug label Jun 15, 2017
jacsonp pushed a commit that referenced this issue Mar 5, 2018
@ghost
Copy link

ghost commented Mar 22, 2018

@jacsonp @guilhermednt podem ajudar nesta issue? 🙏🏼 🍕 🍫
Estamos aplicando o login em cidades do interior de Minas Gerais e está sendo um fator de desestimulo para as pessoas se cadastrarem 😭

@guilhermednt
Copy link

Resumindo (e novas infos) a conversa de ontem no Telegram:

Existem dados embasando esse sentimento? No RS temos mais de 280.000 usuários, centenas de atendimentos todos os meses e, até onde eu sei, 0 reclamações em relação a senha. Pode ser que a baixa adoção seja por outro motivo que ainda não conhecemos. Certa vez estimamos que ~60% dos usuários estavam desistindo de um serviço porque esse serviço faz uma validação adicional dos dados fornecidos e solicita dados adicionais, mas o suspeito, pra variar, foi o Login Cidadão, até mostrarmos que os usuários estavam fazendo todas etapas do LC com quase nenhuma dificuldade... hehehe

Hoje a única diferença entre o Login Cidadão e o Google, por exemplo, é que no LC pedimos que tenha no mínimo 1 número e 1 letra na senha. A seguir temos uma tabela comparando alguns serviços grandes:

Serviço Tamanho mínimo Regra Extra Exemplo Aceito Exemplo Não Aceito
Google 8 chars Algumas exigências não documentadas. Combinações muito simples não são aceitas. ahbgcfde abcdefgh
Facebook 6 chars Algumas exigências não documentadas ahbgcfde abcdefgh
Twitter 6 chars Similar ao Facebook ahbgcfde abcdefgh
Outlook 8 chars No mínimo duas maiúsculas, minúsculas, números ou especiais aaaaaaAA aaaaaaaa

No Login Cidadão, o Bundle que foi usado para implementar isso não é muito configurável, se alguém quiser mandar um PR ou sugerir outro Bundle seria excelente, mas eu particularmente não tenho como priorizar essa demanda pela PROCERGS sem evidências de que isso está sendo um problema (de preferência no RS) e por fora da PROCERGS estou com muito pouco tempo.

As configurações de senha são definidas aqui. O ideal é que fosse possível configurar no config.yml que possamos parametrizar via parameters.yml.

Uma solução excelente que foi dada por Alberto Souza é apenas avisar o usuário que a senha não está boa e deixar que a pessoa decida se quer continuar mesmo assim. Claro que ainda será necessário validar um tamanho mínimo para mitigar brute force mas os outros requisitos eu acredito que deveriam ser configuráveis por cada instância.

Obs.: A única vez que tivemos problemas foi quando tentamos usar Pathwell Topologies, mas isso é configurável (check_pathwell_topologies)

@uira uira added this to the Radar milestone Mar 29, 2018
@uira uira modified the milestones: Radar, Ben Jul 6, 2018
@uira uira added the to do label Jul 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants