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

feat: Add authentication protection to books creation #11

Merged
merged 2 commits into from
May 7, 2021

Conversation

B-Dominguez
Copy link
Contributor

En esta PR se protegen las rutas de creación de libros de forma que solo pueden ser accedidas por un usuario autenticado. Esto permite ejemplificar la protección de rutas con middlewares.

Para lograr lo descrito, se creó un middleware (en un nuevo directorio para middlewares) que revisa si hay un currentUser guardado en el ctx.state. Con él se protegen todas las rutas de creación de libros (formularios y create's). En caso de un intento de acceder a una ruta no autorizada, se levanta el 401, que es manejado con un middleware para redirigir al formulario de login.

Adicionalmente, se condiciona el rendereo de los enlaces a formularios de creación de libros a que haya un usuario autenticado.

@B-Dominguez B-Dominguez requested a review from sivicencio May 7, 2021 04:37
Copy link
Contributor

@sivicencio sivicencio left a comment

Choose a reason for hiding this comment

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

Sólo dejé una mini sugerencia.

Co-authored-by: Sebastián Vicencio <[email protected]>
@B-Dominguez B-Dominguez merged commit daeac92 into main May 7, 2021
@B-Dominguez B-Dominguez deleted the feat/add-routes-protection branch May 7, 2021 14:01
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