-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix EXIF GPS metadata extraction on mobile #360
Conversation
related exifr issue: MikeKovarik/exifr#64 |
Ugh, it looks like the package I chose to use might have a bug that causes this value to show up for ArrayBuffers: pvorb/node-md5#49 |
If I switch to object-hash, I get a sha1 of |
It seems like there's still problems with EXIF metadata on an iPad running iOS 16.5, copy/pasting details from slack so I can still find them after slack deletes the messages: https://reportedcab.slack.com/archives/C9VNM3DL4/p1687400617485609
|
iOS seems to have stopped stripping EXIF GPS metadata as of version 17.1.2, see https://reportedcab.slack.com/archives/C9VNM3DL4/p1704407110590749:
|
This fixes #361. Here's the whole saga:
Here's an example image that works on desktop, but not mobile: https://photos.app.goo.gl/AcEfSrYTMqgp6JfX8
Using
eruda
to see the JS console with this branch, I see this on desktop:but this on mobile:
Seems like this issue might be within the
exifr
library: https://github.com/MikeKovarik/exifrTo rule out the possibility that the EXIF metadata is getting stripped when uploaded from my phone, I logged out the md5sum of the image right before parsing the metadata, and it is
441018525208457705bf09a8ee3c1093
on both desktop and mobile, yet theGPSLatitude
andGPSLongitude
arrays are[NaN, NaN, NaN]
on mobile, but[40, 43, 1.27]
and[73, 59, 36.83]
on desktop, respectively.related exifr issue: MikeKovarik/exifr#64
Ugh, it looks like the package I chose to use might have a bug that causes this value to show up for ArrayBuffers: pvorb/node-md5#49
If I switch to object-hash, I get a sha1 of
b287c307cd89cfd2ce779b975b961b4824f2714a
on desktop, but it crashes on my phone, so I guess I need something less resource-intensive...Seems like Android is stripping GPS from EXIF after all, when the
<input>
element has anaccept
attribute specifying images, but removing the attribute does the trick for me.