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

OT291-2_Middleware_Usuarios_autenticados ok #37

Merged
merged 3 commits into from
Sep 23, 2022

Conversation

sergio17sa
Copy link
Collaborator

  • Middleware para autenticacion de usurios ok
  • Cambié un Throw error en el metodo decryptJWT por un res.status(401).json(error) para conocer el error en la respuesta

const decryptToken = Token.decryptJWT(req, res);

if (!decryptToken) {
return;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Debería devolver algo indicando que no se puede continuar.

@sergio17sa
Copy link
Collaborator Author

  • Corregida observación sobre los errores, el metodo veryfyJWT ya no responde con res y el middleware responde con el error correspondiente
  • Los cambios se realizaron en el commit "Middleware to verify user response errors ok", el siguiente commit "middleware response errors ok" fue para corregir una importacion y parametro que no estaba usando

@soyarielruiz soyarielruiz merged commit eb849f4 into development Sep 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants