[Security] Error leakage in admin.py + no global exception handler#2884
[Security] Error leakage in admin.py + no global exception handler#2884sanrishi wants to merge 1 commit into
Conversation
- Add global Exception handler to main.py returning generic 500 JSON
- Replace bare except:pass with proper logging in admin.py endpoints
- Replace silent return []/{}/True with proper HTTPException raises
- All admin endpoints now log errors instead of swallowing them
Fixes ritesh-1918#2871
|
Someone is attempting to deploy a commit to the ritesh Team on Vercel. A member of the Team first needs to authorize it. |
|
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 |
Adds global exception handler to main.py. Replaces bare except:pass in admin.py with proper logging and HTTPException raises. Fixes #2871