geekcom / validator-docs

Validação de CPF, CNPJ, CNH, NIS, DDD, Título Eleitoral e Cartão Nacional de Saúde com Laravel.
MIT License
565 stars 96 forks source link

Validação de CNH errada #53

Closed MrEko closed 4 years ago

MrEko commented 4 years ago

O validador implementado não cobre todos os casos corretamente. Exemplo real que chegou pra mim hoje: 04463004100.

O validador do 4devs usado como referência do projeto indica falso. O validador em: https://www.treinaweb.com.br/ferramentas-para-desenvolvedores/validar/cnh Aponta corretamento como verdadeiro.

Vou pegar o algoritmo correto com o detran. E depois faço um pull request.

Mas já fica o alerta, e é a 2ª vez que encontro erro nas implementação desse 4devs. Não são muito confiáveis não.

E acredito que deva ter mais exemplos nos casos de teste, com apenas 1 caso é muito fácil passar um algoritmo problemático.

geekcom commented 4 years ago

Boa @MrEko, obrigado pela ajuda que tem dado na lib, realmente os validadores da 4devs tem decepcionado. Vou escrever novos casos de testes para termos maior cobertura.

geekcom commented 4 years ago

https://github.com/geekcom/validator-docs/pull/54