Skip to content

fix(#23): reemplazar currentUserId hardcodeado 1L por usuario autenticado

federico.dominguez requested to merge fix/23-currentUserId-hardcodeado into develop

Cambios realizados

  • Inyectado UserRepository en StudentMedicalController
  • Agregado método helper getCurrentUserId() que obtiene el username desde SecurityContextHolder y busca el User en BD para obtener su ID real
  • Reemplazado Long currentUserId = 1L por getCurrentUserId() en:
    • updateHealthRecord()
    • uploadMedicalFile()
  • Tests: StudentMedicalControllerIntegrationTest (2/2) pasan sin modificaciones

Solución técnica

Se implementó la opción más simple y de menor riesgo: resolver el ID del usuario autenticado a partir del SecurityContext existente, sin necesidad de modificar el JWT ni el sistema de autenticación.

Merge request reports

Loading