Rotation and flip bug? about jpeg-autorotate

[johansatge]

Found the issue (pull request). When flipping the pixels to fix orientations 5 and 7, the destination width and height were wrong. That's why the destination image was broken.

The fix has been pushed in [email protected].

Let's close this. Please feel free to reopen if you still have issues. Thanks again for the report!

[dlednik]

@johansatge any idea what is going on? I need fix for this asap and are willing to fix the source code my self if you give me some pointers where to start 👍

[dlednik]

quick workaround with sharp library if anyone needs it

if (orientation > 1) {
    if (orientation === 5 || orientation === 7) {
        // Until auto-rotate is fixed
        if (orientation === 5) {
            sharp(filePath).rotate(-90).flip().toBuffer().then( data => {
                fs.writeFileSync(filePath, data);
            }).catch( err => {
                Helper.returnError(res, err, 400, `api_v1_photo:get/${req.params.orderId}/${req.params.photoId}`);
            });
        } else {
            sharp(filePath).rotate(90).flip().toBuffer().then( data => {
                fs.writeFileSync(filePath, data);
            }).catch( err => {
                Helper.returnError(res, err, 400, `api_v1_photo:get/${req.params.orderId}/${req.params.photoId}`);
            });
        }
    } else {
        fs.writeFileSync(filePath, buffer);
    }
}

[johansatge]

Hi,

thanks a lot for reporting this.

I reproduce too with the images from the /test/samples directory. It looks like the issue has been here for some time, unfortunately.

I had a quick look in src/transform.js and didn't see what's wrong. I will analyze the issue deeper in the next days. Before next week hopefully.

Note, this issue demonstrates that the existing tests are clearly not covering this module's features correctly. I have opened #12 to try to address this.