Cutlery/jellyfin
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2025-06-04 22:24:35 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix direct play

Browse Source 
The SupportsDirectStream is a little bit misleading as it actually means "Supports Direct Play"

Signed-off-by: gnattu <gnattuoc@me.com>
This commit is contained in:
gnattu 2024-05-05 11:21:10 +08:00
parent a16d3d4887
commit f840d9b60f
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Jellyfin.Api/Controllers/UniversalAudioController.cs
View File

@ -174,11 +174,10 @@ public class UniversalAudioController : BaseJellyfinApiController
return Redirect(mediaSource.Path); 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 // 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; 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 // hls segment container can only be mpegts or fmp4 per ffmpeg documentation
// ffmpeg option -> file extension // ffmpeg option -> file extension