NW6 | Error handling for get videos #23
Conversation
✅ Deploy Preview for watch-next-cyf ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
test (local changes) for .env
RbAvci
force-pushed
the
ZP/Error-Handling-for-Get-Videos
branch
from
606f218 to
f1340dd
Compare
| } catch (error) { | ||
| res | ||
| .status(500) | ||
| .json({ success: false, error: "Could not connect to the database" }); |
There was a problem hiding this comment.
It's good to see you doing some error handling and sending an error message! Question for you: Are exceptions always because the database can't be connected to?
They are likely to be, of course, but that doesn't mean they always are. Imagine, for instance, that you deleted the videos table by accident. This API would say it could not connect to the database which could lead to you spending hours troubleshooting the DB connection. It doesn't matter so much here but it's a good habit. If you have time try to implement it.
You can use examples in the the link to identify the type of error and return a custom error or you can take the message and class out of the error object and return these.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/try...catch#conditional_catch_blocks
No description provided.