Skip to content

security: fix CSRF token validation alerts - #20

Merged
Codewriter90x merged 1 commit into
developmentfrom
security/fix-csrf-codeql-alerts
Jul 10, 2026
Merged

security: fix CSRF token validation alerts#20
Codewriter90x merged 1 commit into
developmentfrom
security/fix-csrf-codeql-alerts

Conversation

@Codewriter90x

Copy link
Copy Markdown
Member

Summary

  • stop API JWT authentication from accepting authCookie as a bearer-token fallback
  • mark stateless API controllers as anti-forgery exempt for CodeQL while keeping bearer-token auth
  • add anti-forgery token propagation for WebApp internal POST/AJAX endpoints
  • route session refresh through same-origin WebApp endpoint protected by anti-forgery
  • add regression coverage for cookie-only JWT API requests

Verification

  • git diff --check
  • dotnet build OpenCashFlow.sln --configuration Release --no-restore
  • dotnet test OpenCashFlow.sln --configuration Release --no-build
  • docker compose config

@Codewriter90x Codewriter90x self-assigned this Jul 10, 2026
@Codewriter90x
Codewriter90x merged commit 61faab8 into development Jul 10, 2026
1 check passed
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