[PR #547] [CLOSED] Switch to external library for getting image sizes #8375

New Issue

OVERLORD · 2026-02-07T05:43:00+03:00

OVERLORD commented

2026-02-07 05:43:00 +03:00

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/547
Author: @Bond-009
Created: 1/11/2019
Status: ❌ Closed

Base: dev ← Head: imagesize

📝 Commits (2)

e341701 Switch to external libraryfor getting image sizes
b2be514 Cleanup ImageProcessor

📊 Changes

7 files changed (+123 additions, -403 deletions)

View changed files

➖ Emby.Drawing/Common/ImageHeader.cs (+0 -243)
📝 Emby.Drawing/Emby.Drawing.csproj (+4 -0)
📝 Emby.Drawing/ImageProcessor.cs (+114 -155)
📝 Emby.Drawing/NullImageEncoder.cs (+1 -1)
📝 Emby.Server.Implementations/ApplicationHost.cs (+1 -1)
📝 Emby.Server.Implementations/Dto/DtoService.cs (+2 -2)
📝 MediaBrowser.Controller/Drawing/IImageProcessor.cs (+1 -1)

📄 Description

I wrote ImageSize.Net from scratch to replace ImageHeader.cs which
supports only the most common versions of the image formats.
ImageSize.Net support more edge cases and has been heavily testing
(in contrast to ImageHeader.cs which was copied from SO)
Project link: https://github.com/Bond-009/ImageSize.Net

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/547 **Author:** [@Bond-009](https://github.com/Bond-009) **Created:** 1/11/2019 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `imagesize` --- ### 📝 Commits (2) - [`e341701`](https://github.com/jellyfin/jellyfin/commit/e3417019f34fff350b87f2e891dc3df8130239e9) Switch to external libraryfor getting image sizes - [`b2be514`](https://github.com/jellyfin/jellyfin/commit/b2be514286489cedc76c4b1f7c58a4c1258bc4f5) Cleanup ImageProcessor ### 📊 Changes **7 files changed** (+123 additions, -403 deletions) <details> <summary>View changed files</summary> ➖ `Emby.Drawing/Common/ImageHeader.cs` (+0 -243) 📝 `Emby.Drawing/Emby.Drawing.csproj` (+4 -0) 📝 `Emby.Drawing/ImageProcessor.cs` (+114 -155) 📝 `Emby.Drawing/NullImageEncoder.cs` (+1 -1) 📝 `Emby.Server.Implementations/ApplicationHost.cs` (+1 -1) 📝 `Emby.Server.Implementations/Dto/DtoService.cs` (+2 -2) 📝 `MediaBrowser.Controller/Drawing/IImageProcessor.cs` (+1 -1) </details> ### 📄 Description I wrote ImageSize.Net from scratch to replace ImageHeader.cs which supports only the most common versions of the image formats. ImageSize.Net support more edge cases and has been heavily testing (in contrast to ImageHeader.cs which was copied from SO) Project link: https://github.com/Bond-009/ImageSize.Net --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-07 05:43:00 +03:00

OVERLORD closed this issue

2026-02-07 05:43:01 +03:00

OVERLORD referenced this issue

2026-02-07 06:42:53 +03:00

[PR #8375] Fix musl build permissions #11695

Sign in to join this conversation.

Branches Tags

master

release-10.11.z

renovate/skiasharp-monorepo

JPVenson-patch-1

renovate/morestachio-5.x

release-10.9.z

openapi-cache

release-10.10.z

explicit-load

explicit-load-v2

feature/entity-framework

only-count-folder-item

fix/ci-workflow-refactor

release-10.8.z

release-10.7.z

EraYaN-add-management-interface

release-10.6.z

release-10.5.z

release-10.4.z

release-10.3.z

release-10.2.z

release-10.1.0

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#8375