Conversation
vinipet
left a comment
vinipet
left a comment
There was a problem hiding this comment.
cara esse pr ta imenso, antes de abrir pr, tenta colocar varios commits bem nomeados, prq ai eu sei oque é pulavel doque nao não é. vi tmb que vc refatoro bastante codigo so nao sei c essas fatorações tem haver com a parte security que o titulo sugere. vou dar uma olhada mais detalhada nos DESENOVE ARQUIVOS IMENSOS, e pontuar separadamente cada ponto que eu achar importante
There was a problem hiding this comment.
esse arquivo mesmo é refatoração, nao ta ruim, é até bom, mas ai tem que ser um PR exclusivo pra isso, pensa no futuro caso vc queira olhar o historico do repo pra achar onde ta as alterações dessa parte. Vc nao acharia
There was a problem hiding this comment.
vdd vou melhorar isso, acabei colocando tudo no mesmo pacote
There was a problem hiding this comment.
aqui tem um metodo nao finalizado, esse arquivo é praticamente um arquivo de comentario pra lembrar de fazer o metodo dps, isso nao subiria pra produção. imagine o repositorio aqui como produção
There was a problem hiding this comment.
aqui tem um metodo nao finalizado, esse arquivo é praticamente um arquivo de comentario pra lembrar de fazer o metodo dps, isso nao subiria pra produção. imagine o repositorio aqui como produção
There was a problem hiding this comment.
aqui tem um metodo nao finalizado, esse arquivo é praticamente um arquivo de comentario pra lembrar de fazer o metodo dps, isso nao subiria pra produção. imagine o repositorio aqui como produção
There was a problem hiding this comment.
aqui ta uma alteração que gostei bastante, fez uma refatoração no lombok que faz sentido com o pr proposto. percebeu que tava duplicado o bgl, e add as info pra segurança do usuario, isso aqui passaria tranquilo, por mim logico
There was a problem hiding this comment.
aqui eu nao entendi oo motivo dessa interface existir. c todos os atributos são opcionais, prq criar uma lei que os obrigue?
2 participants
Implementei as funcionalidades de login e registro de usuário. Todos os serviços foram validados:
✅ Docker e MySQL testados e aprovados
✅ Insomnia testado e aprovado
✅ Módulo de livros não foi alterado, mas foi testado e segue funcionando normalmente
A partir de agora, todas as rotas exigem um token de autenticação para serem acessadas.
Como obter o token?
Ao fazer login ou registrar uma nova conta, o token será retornado automaticamente no corpo da resposta da requisição. Basta copiá-lo e utilizá-lo nas demais chamadas.