fix(api): propagate database errors correctly with 500 status code#1884
Conversation
|
@panditshubham766-dotcom is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
There was a problem hiding this comment.
Thanks for your first PR on DevTrack! 🎉
A maintainer will review it within 48 hours. While you wait:
- Make sure CI is passing (type-check + lint)
- Double-check the PR description is filled out and the issue is linked
- Feel free to ask questions in Discussions if you need help
If you find DevTrack useful, a ⭐ star on the repo is always appreciated — it helps the project grow and attract more contributors!
e4ff79b
into
Priyanshu-byte-coder:main
|
🎉 Merged! Thanks for contributing to DevTrack. If the project has been useful to you, a ⭐ star on the repo is the easiest way to support it — it helps DevTrack get discovered by more developers. Keep an eye on open issues for your next contribution! |
Summary
Fixed a critical backend issue where several API routes were catching database/query failures but hiding them by returning a
200 OKstatus with empty fallback data. Unhandled exceptions now correctly propagate as a500 Internal Server Errorto prevent the frontend from misinterpreting database failures as successful empty responses.Closes #1792
Type of Change
Changes Made
200error-masking blocks with properNextResponse.jsonerror payloads containing a500status code.200empty states (like empty pins data or unconfigured profiles) so normal application behavior remains uninterrupted.How to Test
/api/local-coding/stats).500 Internal Server Errorpayload instead of a fake200 OK.