Skip to content

Document why CSRF stays disabled in the stateless JWT API [skip deploy] #no-deploy - #122

Merged
SebastianLopezO merged 1 commit into
mainfrom
security/codeql/spring-disabled-csrf-protection
Nov 1, 2026
Merged

SebastianLopezO merged 1 commit into
mainfrom
security/codeql/spring-disabled-csrf-protection

Conversation

@SebastianLopezO

Copy link
Copy Markdown
Member

Caso

  • Origen: CodeQL, alerta #16
    java/spring-disabled-csrf-protection en SecurityConfiguration.java.
  • Severidad: alta (según la regla).
  • Análisis: la protección CSRF sirve cuando el navegador envía la credencial por su cuenta (cookie de sesión). La API
    es sin estado (SessionCreationPolicy.STATELESS), solo acepta el JWT en la cabecera Authorization y no usa cookies
    ni CORS con credenciales (allowCredentials no se activa). Un sitio ajeno no puede añadir esa cabecera a una petición
    del navegador de la víctima, así que el ataque no aplica. Activar CSRF obligaría a la PWA a manejar un token extra sin
    ganancia de seguridad.

Cambio

  • Comentario en la cadena de filtros que explica la decisión y la condición para revertirla: si la sesión pasa algún
    día a una cookie, CSRF debe activarse.

Después de fusionar

  • La alerta se descarta como falso positivo con este análisis.

@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@SebastianLopezO
SebastianLopezO force-pushed the security/codeql/spring-disabled-csrf-protection branch from 35bb5dd to 8666b1f Compare November 1, 2026 03:44
@SebastianLopezO
SebastianLopezO merged commit 52273d7 into main Nov 1, 2026
8 of 9 checks passed
@SebastianLopezO
SebastianLopezO deleted the security/codeql/spring-disabled-csrf-protection branch November 1, 2026 03:47
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.

1 participant