Include AVIF extension for support images #12415
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AVIF is supported by jellyfin, but is currently being filtered out based on the extension. This adds the extension to the list of support file types. Close jellyfin#12412
gnattu
approved these changes
Aug 9, 2024
There was a problem hiding this comment.
I'm personally fine with this change as AVIF is claimed as supported in the release note. But I have to remind that this support is not complete and internal functions like generate lower resolution thumbnail may still not work and will always return original picture, but to fix that is out of the scope of 10.9
nielsvanvelzen
approved these changes
Aug 22, 2024
crobibero
approved these changes
Aug 22, 2024
joshuaboniface
pushed a commit
that referenced
this pull request
Aug 25, 2024
Include AVIF extension for support images Original-merge: 5027e3c Merged-by: crobibero <cody@robibe.ro> Backported-by: Joshua M. Boniface <joshua@boniface.me>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Jellyfin will currently process avif files (as demonstrated by renaming a valid
.aviffile to.jpg, and jellyfin indexing and displaying it perfectly) but currently does not try processing for some reason.Changes
Adds the AVIF extension to the list of supported image types.
Issues
Close #12412