immich-app/immich
1
0
Fork 0
mirror of https://github.com/immich-app/immich.git synced 2026-02-27 19:08:16 +03:00
Code Issues 783 Packages Projects Releases 12 Wiki Activity

[PR #11610] fix(server): video duration extraction #13291

New Issue
Closed
opened 2026-02-05 15:12:11 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2026-02-05 15:12:11 +03:00
Owner
Copy Link

Original Pull Request: https://github.com/immich-app/immich/pull/11610

State: closed
Merged: Yes

Extracting the video duration based on EXIF data has proven to be unreliable. Exiftool has 7 different ways for calculating the Duration value, and if any of those are missing or invalid the duration will also be wrong. Using ffprobe is a more reliable solution to determine the video duration.

Using an MKV test file the duration is now correctly extracted as 00:01:27.336, whereas it was previously 24:15:36.000. This should fix #7312 and fix #8244, though I can't verify that myself

**Original Pull Request:** https://github.com/immich-app/immich/pull/11610 **State:** closed **Merged:** Yes --- Extracting the video duration based on EXIF data has proven to be unreliable. Exiftool has [7 different ways](https://exiftool.org/TagNames/Composite.html) for calculating the `Duration` value, and if any of those are missing or invalid the duration will also be wrong. Using `ffprobe` is a more reliable solution to determine the video duration. Using an [MKV test file](https://github.com/ietf-wg-cellar/matroska-test-files/blob/e6965e5ca666322ed93e2748a10a4f132309e005/test_files/test1.mkv) the duration is now correctly extracted as `00:01:27.336`, whereas it was previously `24:15:36.000`. This should fix #7312 and fix #8244, though I can't verify that myself
OVERLORD added the pull-request label 2026-02-05 15:12:11 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
accessibility
changelog:enhancement
changelog:security
changelog:skip
changelog:translation
cli
date-time
dependencies
documentation
external-library
format
good first issue
mobile-beta
mobile-beta
mobile-beta
needs-answer
nice to have
pull-request
Mirrored from GitHub Pull Request
 sharing
tech-debt
📱mobile
🖥️web
🗄️server
🧠machine-learning
No Label pull-request
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#13291
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?