-
Notifications
You must be signed in to change notification settings - Fork 205
Error logs #76
Comments
I'm open to discussing a PR for this, perhaps using Morgan or a similar lib. |
Not sure what fits the best to have good logging but for example, in 'source/controllers/api/uploads_controller.js',
'err' is not used and the only message I get is that it didn't work out well but if you can even simply 'console.error(`Saving the image to a file failed : ${err}`)', this will let people figure out what just happened. |
That's definitely true for a dev environment where you have the console available — and I fully agree better error messages need to be in place (even if just for the user). Thanks for pointing this out. |
Summary
Can you try to print out error logs to the console when an error occurs? You seem to discard 'err' variables and it's hard to debug with no information.
I tried to upload an image and it failed with no logging. It turned out that I hadn't installed 'gm' command but it could be better if 'err' content was printed in the server as console.error.
Steps to Reproduce
Additional info
The text was updated successfully, but these errors were encountered: