jellyfin/MediaBrowser.Controller/Drawing, branch v12.0-rc5Unnamed repository; edit this file 'description' to name the repository.
https://git.b0n.dev/jellyfin/atom?h=v12.0-rc52026-08-10T20:10:08ZUpdate MediaBrowser.Controller/Drawing/ImageHelper.cs2026-08-10T20:10:08ZCody Robiberocody@robibe.ro2026-08-10T20:10:08Zurn:sha1:82ecfde2f7bdd9d91ec403749c124ef96f64cea3
Co-authored-by: Tim Eisele <Tim_Eisele@web.de>Stop image endpoints from upscaling beyond the source resolution2026-08-07T15:33:37Zvavalleevavallee@protonmail.com2026-08-07T15:33:37Zurn:sha1:e120b7f2dd986d7f07f6814b6d987bafd46baab8
ImageHelper.GetNewImageSize passed the caller-supplied width/height straight
through to SkiaEncoder.EncodeImage, which allocates an SKImageInfo of exactly
that size. Nothing bounded those values against the source image, so a request
like Items/<id>/Images/Primary?width=23100&height=23100 made the server allocate
and resample a 23100x23100 surface from, say, a 600x336 poster: the reporter
measured 100% of a core for 10-15 minutes and 6-12 GB resident per request.
The item images endpoints do not require authentication, so any caller who knows
an item id can trigger this, and varying the size by one pixel misses the cache
every time.
Add DrawingUtils.ScaleDownToFit, which scales a size down uniformly until it
fits inside a bounding box and returns it unchanged if it already does, and
apply it in GetNewImageSize against the original image dimensions. Requests
that ask for more pixels than the source now get the source resolution back,
scaled to the requested aspect ratio. Downscaling paths are untouched, and
DrawingUtils.Resize keeps its existing behaviour for the transcoding callers in
EncodingJobInfo and StreamInfo, which legitimately size video output.
ResizeFill already refused to upscale; this makes width/height consistent
with fillWidth/fillHeight.
Fixes #17056.
Fixed namespaces2025-03-25T15:45:00ZJPVensongithub@jpb.email2025-03-25T15:45:00Zurn:sha1:42bdb22bfb690a6af37d70f12844881d884927b1Naming refactoring and WIP porting of new interface repositories2024-10-09T09:53:39ZJPVenson6794763+JPVenson@users.noreply.github.com2024-10-09T09:53:39Zurn:sha1:be48cdd9e90ed147c5526ef3fed0624bcbad7741Remove some unused drawing code2024-02-06T15:11:47ZPatrick Barronbarronpm@gmail.com2024-02-06T15:09:51Zurn:sha1:4dd2ed8fb7bbd825995e7ec4aead11d6d9728a19Merge branch 'master' into trickplay2023-10-19T02:27:05ZNick20588554+nicknsy@users.noreply.github.com2023-10-19T02:27:05Zurn:sha1:cd662506a1f63f9b20e7f5caa9b671eb3d71ea5aJellyfin.Drawing minor improvements2023-10-08T15:10:04ZBond_009bond.009@outlook.com2023-10-05T21:57:11Zurn:sha1:c707baed8366cd44ea80713cf93e5f92971815bf
Reduce duplicate/dead code
Merge pull request #10056 from Bond-009/nullable22023-09-12T09:09:20ZBond-009bond.009@outlook.com2023-09-12T09:09:20Zurn:sha1:3087881d93591ccf17af35021e0d286756489221Enable nullable for more files and add tests2023-08-22T16:11:34ZBond_009bond.009@outlook.com2023-08-22T16:09:31Zurn:sha1:d92e9ae85e41fef981729f544bfd6df2c052a712
Adds basic tests for FFProbeVideoInfo.CreateDummyChapters
Fixed error message CreateDummyChapters instead of reporting the total minutes it only reported the minute component
Enable nullable for more files2023-07-29T19:35:38ZBond_009bond.009@outlook.com2023-07-29T19:35:38Zurn:sha1:5677566a41638f4c62f107f3540363457c099019