You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Follow up remaining CodeQL security alerts outside #226 #227
Follow-up to #226. This issue tracks the remaining active CodeQL findings intentionally outside the targeted password-persistence, API/social rate-limit, and CSRF remediation.\n\nScope:\n- verify CodeQL closes the former user-controlled logout redirect after the CSRF companion change, and investigate only if it remains;\n- replace the permissive CORS configuration in server/index.js with an explicit origin allowlist;\n- add evidence-backed rate limiting for the remaining authorization and filesystem-serving routes (server/auth/index.js, server/index.js) and the server/api/friends.test.js fixture if it remains in CodeQL scope.\n\nAcceptance criteria:\n- every remediation has a focused regression test or runtime verification;\n- CodeQL is rerun and the finding is closed by code, not suppression, dismissal, or query changes;\n- CORS behavior is checked against the deployed frontend/auth callback configuration before production rollout (#176).\n\nDo not broaden this issue into database/schema or email-related work.
CSRF-library follow-up: #228 uses lusca.csrf() because the repository’s current CodeQL engine explicitly recognizes it. lusca 1.7.0 is a legacy/inactive dependency, so this issue must reassess and migrate to a maintained CSRF library when the CodeQL version/rule support permits equivalent detection. Keep real token-validation coverage; do not suppress or dismiss the finding.
Activity
coder13 commentedon Jul 14, 2026
CSRF-library follow-up: #228 uses
lusca.csrf()because the repository’s current CodeQL engine explicitly recognizes it.lusca1.7.0 is a legacy/inactive dependency, so this issue must reassess and migrate to a maintained CSRF library when the CodeQL version/rule support permits equivalent detection. Keep real token-validation coverage; do not suppress or dismiss the finding.