Skip to content

Conversation

@crlian
Copy link

@crlian crlian commented Feb 1, 2026

Summary

  • Se agregó la anotación @PreAuthorize al endpoint /attendees/profile

Why?

  • El endpoint /attendees/profile era accesible sin autenticación. Un usuario podía acceder al endpoint sin token y causar un NullPointerException

Changes Made

Testing

  • Acceso sin autorización al endpoint /attendees/profile ahora devuelve 403
  • Los request con autenticación todavía siguen funcionando
  • El código compila sin errores

Notas

Este es un ejemplo de PR con las notaciones que seguiremos.

  • Como estructurar los commits, ramas y descripción de los PR
  • Como se dejará los comentarios del code review
  • Buenas prácticas del flujo Git/Github

Copy link
Author

@crlian crlian left a comment

Choose a reason for hiding this comment

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

Revisar como se siguió el flujo de PR:

  • Conventional commits
  • Un PR con Summary, Why, Changes, Testing
  • Ahora un endpoint protegido requiere autenticación

Este es un ejemplo de como los PR debería verse

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