mirror of
https://github.com/jellyfin/jellyfin.git
synced 2026-05-04 18:09:12 +03:00
DNLA - Play Function is not available from new samsung tv. #2349
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @righttime on GitHub (Dec 6, 2020).
Originally assigned to: @BaronGreenback on GitHub.
Describe the bug
With DLNA.
List and thumbnails are shown at TV, but It is not able to play video.
I've installed it now, so I don't know what to do now.
but I've done what I can, but I can't make it.. please help :)
From mobile, and PC(Chrome) it is working fine, I can play the videos.
but my tv looks like doesn't have a token for...auth. (TV problem?)
System (please complete the following information):
To Reproduce
Just play video from samsung tv via dlna
Expected behavior
Play!
Logs
Screenshots
Additional context
@righttime commented on GitHub (Dec 6, 2020):
and the profile ("Samsung Smart TV") is just selected from profile list and not changed at all.
@BaronGreenback commented on GitHub (Dec 7, 2020):
@crobibero This uses the API /videos/b5145bdf-d238-928d-db87-65d5cb85b4d1/stream.mkv
As the API is now defined as /stream, could this be the issue?
@crobibero commented on GitHub (Dec 7, 2020):
That endpoint still exists
a7b461adb4/Jellyfin.Api/Controllers/VideosController.cs (L586)@BaronGreenback commented on GitHub (Dec 8, 2020):
Could be fixed by https://github.com/jellyfin/jellyfin/pull/4724
@marius-luca-87 commented on GitHub (Mar 1, 2021):
I have tested on Samsung Q6FNA with Jellyfin 10.7.0-RC4 and DLNA play functionality is still broken.
When debugging on a local build I found a possible issue at:
153021eec6/Jellyfin.Api/Controllers/VideosController.cs (L385)var isHeadRequest = Request.Method == System.Net.WebRequestMethods.Http.Head;For the Samsung TV the Request.Method seems to always be System.Net.WebRequestMethods.Http.Head.
Hardcoding isHeadRequest to false allows videos to play from DLNA.
@marius-luca-87 commented on GitHub (Mar 1, 2021):
Server response to HEAD request retrieved from Wireshark:
10.6.4
10.7.0-rc4
@BaronGreenback commented on GitHub (Mar 1, 2021):
Looking at the 10.6 code to see what's changed.
@BaronGreenback commented on GitHub (Mar 1, 2021):
@marius-luca-87: Are you able to test PR https://github.com/jellyfin/jellyfin/pull/5335 ?
My educated guess (haven't been able to cause the same issue here, so can't be 100% sure) is that the problem is with the content-length being zero.
The PR above should re-populate it with the value.
@marius-luca-87 commented on GitHub (Mar 1, 2021):
I have run a test with the proposed changes and the content length remains 0.
I haven't looked into the documentation for FileContentResult but it might be overwriting the Content-Length response header based on the Array.Empty() provided.
@BaronGreenback commented on GitHub (Mar 1, 2021):
@marius-luca-87 I've just committed a change to the PR. Could you test again please?
Am unable to test here - as I have other issues with my TV playback.
@marius-luca-87 commented on GitHub (Mar 1, 2021):
Playback seem to work ok with the latest changes. In my case this issues seems to be resolved.
@BaronGreenback commented on GitHub (Mar 1, 2021):
@marius-luca-87 Could you try once more please? Have removed the content-length altogether on this one - as per the specs.
@marius-luca-87 commented on GitHub (Mar 1, 2021):
I have tested the changes with the Content-Length removed (verified in Wireshark) and can confirm that for the Samsung Q6FNA the issue is resolved.
@BaronGreenback commented on GitHub (Mar 1, 2021):
Thanks for your help testing.