immich-app/immich
1
0
Fork 0
mirror of https://github.com/immich-app/immich.git synced 2026-02-05 00:30:57 +03:00
Code Issues 11 Packages Projects Releases 10 Wiki Activity

[BUG] harden the exif extraction process more #678

New Issue
Closed
opened 2026-02-04 21:52:11 +03:00 by OVERLORD · 2 comments
OVERLORD commented 2026-02-04 21:52:11 +03:00
Owner
Copy Link

Originally created by @Snuupy on GitHub (Feb 9, 2023).

The bug

https://discord.com/channels/979116623879368755/994044917355663450/1073126252417912843

parse any integer so that if this case happen it will default to 0 so other info can still be inserted

symptom: file contains EXIF data (possibly invalid), no EXIF data at all is displayed within immich

fix should:

  • display exif data even if other fields are not valid
  • add better logs there to make it more obvious why it failed

Version of Immich Server

v1.45

Version of Immich Mobile App

v1.45

Platform with the issue

  • Server
  • Web
  • Mobile

Reproduction steps

  1. grab file on discord
  2. import into immich
  3. see that no exif data is shown
  4. open same file in irfanview, press e, see that exif data is shown
Originally created by @Snuupy on GitHub (Feb 9, 2023). ### The bug https://discord.com/channels/979116623879368755/994044917355663450/1073126252417912843 > parse any integer so that if this case happen it will default to 0 so other info can still be inserted symptom: file contains EXIF data (possibly invalid), no EXIF data at all is displayed within immich fix should: - display exif data even if other fields are not valid - add better logs there to make it more obvious why it failed ### Version of Immich Server v1.45 ### Version of Immich Mobile App v1.45 ### Platform with the issue - [X] Server - [ ] Web - [ ] Mobile ### Reproduction steps 1. grab file on discord 2. import into immich 3. see that no exif data is shown 4. open same file in irfanview, press `e`, see that exif data is shown
OVERLORD added the 🗄️server label 2026-02-04 21:52:11 +03:00
OVERLORD commented 2026-02-04 21:52:16 +03:00
Author
Owner
Copy Link

@uhthomas commented on GitHub (Jul 9, 2023):

Will be resolved by #2908.

@uhthomas commented on GitHub (Jul 9, 2023): Will be resolved by #2908.
OVERLORD commented 2026-02-04 21:52:19 +03:00
Author
Owner
Copy Link

@bo0tzz commented on GitHub (Sep 18, 2023):

Fixed by #2908

@bo0tzz commented on GitHub (Sep 18, 2023): Fixed by #2908
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
accessibility
changelog:enhancement
changelog:security
changelog:translation
cli
date-time
documentation
external-library
format
good first issue
needs-answer
nice to have
sharing
tech-debt
📱mobile
🖥️web
🗄️server
🧠machine-learning
No Label 🗄️server
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#678
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?