search

uspdev / web-ldap-admin

https://gitlab.uspdigital.usp.br/uspdev/web-ldap-admin
3 stars 9 forks source link

Erro no teste de senhas iguais #159

Open ezanon opened 4 months ago

ezanon commented 4 months ago

O teste requer no mínimo 8 caracteres. Se o aluno coloca senha com 8 caracteres, dá erro, dizendo que tem que ter entre 8 e 20, mas tem 8! Se colocar 9 caracteres, aí funciona. WhatsApp Image 2024-03-11 at 12 07 14

alecostaweb commented 4 months ago

Um colega da Odonto, detectou o mesmo problema, mas só depois de subir para php8.1. No caso, foi resolvido mudando a verificação de 'min:8|max:20' para 'min:8', 'max:20'. Isso no arquivo web-ldap-admin/app/Http/Controllers/LdapUserController.php, linha 227. Minha sugestão é já aplicar no release atual. Podem testar e mandrem PR

ezanon commented 4 months ago

@alecostaweb Funcionou com essa correção. Alunos testaram já. Rodando com o php8.2 aqui. Obrigado! Realmente será interessante aplicar essa correção o mais breve possível.

Um colega da Odonto, detectou o mesmo problema, mas só depois de subir para php8.1. No caso, foi resolvido mudando a verificação de 'min:8|max:20' para 'min:8', 'max:20'. Isso no arquivo web-ldap-admin/app/Http/Controllers/LdapUserController.php, linha 227. Minha sugestão é já aplicar no release atual. Podem testar e mandrem PR