Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JPEG marker not found error when using recursive #43

Closed
joeyhoer opened this issue Mar 26, 2014 · 6 comments
Closed

JPEG marker not found error when using recursive #43

joeyhoer opened this issue Mar 26, 2014 · 6 comments

Comments

@joeyhoer
Copy link

When attempting to use the recursive command line flags, I am receiving a generic JPEG marker not found error. This isn't happening in every use case, I believe it's only occurring when a corrupted JPEG is encountered.

EDIT: Upon further inspection, the JPEG does not appear to be corrupted. The header is 24bit N Exif IPTC ICC Adobe JFIF

@toy
Copy link
Owner

toy commented Mar 30, 2014

@joeyhoer Error comes from image_size, I think it will be better to make image_optim ignore files for which type can't be determined and show warning. Is it possible that you share this jpeg to understand if the problem is in the file or in image_size and if gem can be enhanced?

@toy
Copy link
Owner

toy commented Apr 6, 2014

@joeyhoer Please try v0.13.
It will be very helpful if you can share jpeg file causing problems, you can open an issue for image_size gem.

@toy toy closed this as completed Apr 6, 2014
@joeyhoer
Copy link
Author

Unfortunately, upgrading in did not resolve the issue. The attached image is one example of an image which produces this error.

The image is not explicit, however it may be considered NSFW. Image

@toy
Copy link
Owner

toy commented Jun 24, 2014

@joeyhoer Thanks for the image. Already problematic one: identify for this image warns identify: Corrupt JPEG data: 12063 extraneous bytes before marker 0xe2 'test.jpg' @ warning/jpeg.c/JPEGWarningHandler/351.

@toy
Copy link
Owner

toy commented Jun 24, 2014

@joeyhoer Please check with image_size v1.3.1

@joeyhoer
Copy link
Author

joeyhoer commented Jul 2, 2014

Working well, as far as I can tell.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants