[HIGH] Wire SecurityHeadersMiddleware into main.py — app missing CSP, HSTS, X-Frame-Options (Fixes #2892)#2895
Conversation
…1918#2892) - Added import and registration of SecurityHeadersMiddleware for CSP, HSTS, X-Frame-Options, X-Content-Type-Options, and other security headers - Removed duplicate SUPABASE_URL / SUPABASE_SERVICE_KEY env reads - Previously the middleware class was fully implemented but never wired in
|
@namann5 is attempting to deploy a commit to the ritesh Team on Vercel. A member of the Team first needs to authorize it. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Description
The fully-implemented \SecurityHeadersMiddleware\ in \�ackend/security_middleware.py\ was never wired into the FastAPI application. This meant the app served all responses without:
Changes
Closes #2892