Add callback call in the resizer function. #38
Conversation
…erwise the exception might cannot be caught outside.
| @@ -41,6 +41,7 @@ extensions = ['.jpg', '.jpeg', '.png']; | |||
| resizer = (options, callback) => | |||
| jimp.read(options.srcPath, (err, file) => { | |||
| if (err) { | |||
| if(done) return done(file, err); | |||
There was a problem hiding this comment.
If we're returning early from the function, we don't need to throw the error.
|
Also, the reason why Travis is failing is because of formatting. You can run |
|
I just pushed some fixes that should take care of this. Would you mind testing your code with latest master? |
|
hi honza, I just checked your fix, it works in my project. It will not make the app crash after trying to creating thumbnail for a fake image, but the error was just hidden underground as you are calling the internal callback but not the customized callback. Anyway, it's good enough for my project so far. Thanks for your fix. By the way, the CI shows there are conflicts and I just try to fixed, not sure how to restart the ci so please help to check. |
Thank the team first as this library is useful in one of my project.
But when using it, I encountered one problem, when using in expressjs, which is when the user upload a fake picture (like after changing the ext of another non-pic file) and the server try to make a thumbnail of it, there will be an exception (unkown MIME type) in the jimp which will cause the resizer function to throw out the exception. But in expressjs, I cannot catch the exception which will cause the process terminate.
I am not sure whether I could find a better solution in expressjs side, but for now I just folk this project and add the callback in resizer function so that I can process the exception in the expressjs.
It will be appreciated if someone can provide a better solution without changing this lib!