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

DefaultImageHeaderParser fails to determine correct orientation #3944

Open
grote opened this issue Oct 23, 2019 · 7 comments
Open

DefaultImageHeaderParser fails to determine correct orientation #3944

grote opened this issue Oct 23, 2019 · 7 comments

Comments

@grote
Copy link

grote commented Oct 23, 2019

Glide Version: 4.10.0

Integration libraries: none

Device/Android Version: Pixel 2 XL Android 10

Issue details / Repro steps / Use case background: Glide (but in its defense also my browser) fails to determine the orientation of this image:

rotate-test

DefaultImageHeaderParser#getOrientation() returns UNKNOWN_ORIENTATION.

However, Android's gallery app and my Linux image viewer apps display the orientation correctly. Here's jhead output:

Resolution   : 800 x 600
Orientation  : rotate 90
Jpeg process : Progressive
JPEG Quality : 93

So the orientation is in there, but maybe it is hard to parse?

@photospringdev
Copy link

We are having the same issue - some with Google Pixel but also with Samsung galaxy. Gallery app on our device displays correctly.
I cannot publicly share the photos that have issues but if you email support@ photospring I can email you the photos.

@Apu-RR
Copy link

Apu-RR commented Mar 11, 2021

Same issue here, will this be fixed anytime soon?

@quite
Copy link

quite commented Apr 26, 2021

Same issue on Nokia 5.3 (2020) with Android 10. Seems consistent for all photos taken in portrait orientation.
exiv2 says: Exif.Image.Orientation Short 1 right, top
Attaching one.

IMG_20210426_155126

@grote
Copy link
Author

grote commented May 4, 2021

Glide (but in its defense also my browser) fails to determine the orientation of this image

Update: Firefox now correctly identifies the orientation. I guess it is time for glide to follow.

@quite
Copy link

quite commented May 5, 2021 via email

@quite
Copy link

quite commented May 5, 2021

I did manage to install Firefox 70. Both it and recent Firefox 88 displays the above photo correctly as a portrait.

@grote
Copy link
Author

grote commented May 5, 2021

Sorry, I don't remember what Firefox version I had installed in 2019 when I reported this issue. But good to know that even Firefox 70 fixed the problem.

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

No branches or pull requests

5 participants