-
Notifications
You must be signed in to change notification settings - Fork 8
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
Tratamento de Erros Backend #117
Conversation
refactor:change error handling
a7c52c6
to
9fe3659
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trabalho muito bem feito, parabéns! Ainda não consegui testar com o frontend, mas assim que sobrar um tempo eu testo. Encontrei alguns probleminhas no codigo que falo mais abaixo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
56e4852
to
e21a00e
Compare
e21a00e
to
c4b2efe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bom, pelos vários testes que fiz, o middleware não está funcionando. Um exemplo fácil de se testar é mandar uma requisição para uma rota que precisa estar autenticado com um token invalido. Deveria responder com o UnauthorizedError, mas não é enviado nenhuma resposta. Acredito que o express não está linkando com o errorHandler
, pois não está nem entrando nesse metodo.
utils/validation/middlewares/authFirebase.js
no metodo isAuthenticated
c4b2efe
to
b01c7c5
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Descrição
Criação de classes que representam os erros HTTP e de um middleware global para retornar os erros adequados.
Resolve (Issues)
Tratamento de Erros Backend
Tarefas gerais realizadas