From f1c000e4fc87051b291993cfab4ebf74dc94935b Mon Sep 17 00:00:00 2001 From: gnattu Date: Sun, 5 May 2024 11:21:10 +0800 Subject: [PATCH] Fix direct play The SupportsDirectStream is a little bit misleading as it actually means "Supports Direct Play" Signed-off-by: gnattu --- Jellyfin.Api/Controllers/UniversalAudioController.cs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/Jellyfin.Api/Controllers/UniversalAudioController.cs b/Jellyfin.Api/Controllers/UniversalAudioController.cs index 1305d14176d..3286105e39c 100644 --- a/Jellyfin.Api/Controllers/UniversalAudioController.cs +++ b/Jellyfin.Api/Controllers/UniversalAudioController.cs @@ -174,11 +174,10 @@ public class UniversalAudioController : BaseJellyfinApiController return Redirect(mediaSource.Path); } - // This one is currently very misleading as the SupportsDirectStream is always false + // This one is currently very misleading as the SupportsDirectStream actually means "can direct play" // The definition of DirectStream also seems changed during development - // It used to mean HTTP direct streaming, but now HLS is used even for DirectStream var isStatic = mediaSource.SupportsDirectStream; - if (mediaSource.TranscodingSubProtocol == MediaStreamProtocol.hls) + if (!isStatic && mediaSource.TranscodingSubProtocol == MediaStreamProtocol.hls) { // hls segment container can only be mpegts or fmp4 per ffmpeg documentation // ffmpeg option -> file extension