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

Reading iPhone MOV throws File format could not be determined #344

Open
rn-89 opened this issue Sep 2, 2023 · 4 comments
Open

Reading iPhone MOV throws File format could not be determined #344

rn-89 opened this issue Sep 2, 2023 · 4 comments

Comments

@rn-89
Copy link

rn-89 commented Sep 2, 2023

Starting around iOS15, I've been getting the "File format could not be determined " exception when trying to read the metadata of a video file of a Live Photo. A Live Photo on an iPhone creates an HEIC image file as well as a video file. The video file from that package is what is throwing the exception. I'm attaching a sample video file that should recreate this exception. Tested with .NET 7 running MetadataExtractor 2.8.1.

IMG_8827.mov
@HEIC-to-JPEG-Dev
Copy link

Yes, I get this as well. Apple don’t like Windows :(

I’d use this library if this was fixed, otherwise it’s of no use to me as Apple files are prolific.

I can provide video files if that helps with the development of this capability

@HEIC-to-JPEG-Dev
Copy link

I would add that “throwing an exception” is very bad for performance. If I’m getting the data for a thousand files and 20% are Apple videos, this package hits a wall performance wise. It should return a code instead. Exceptions are for unexpected errors if all goes wrong, not for known, expected things you’re checking for.

@HEIC-to-JPEG-Dev
Copy link

I think Apple are using hevc encoding now

naughtyGitCat added a commit to naughtyGitCat/metadata-extractor-dotnet that referenced this issue Sep 9, 2024
@naughtyGitCat
Copy link

@rn-89 @HEIC-to-JPEG-Dev happens with same error, the header info changes, fix it with an if .... else .... to catch this circumstance

@drewnoakes please take a notice, can you merge my pr?

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

3 participants