Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DLNA fix : Make media visible and playable #5951

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

@BaronGreenback
Copy link
Contributor

@BaronGreenback BaronGreenback commented May 1, 2021

Some devices / software do not show media if they cannot play use container, stopping the possibility of it be transcoded.
VLC and Hisense TV's both do this - i'm sure there are others.

This change returns all media via DLNA with the extension .ts. The original file extension is passed as a parameter.

Once the request to stream has been received, this correct file extension is used for transcoding selection / playback.

Have tried without using a file extension - but the same issue occurs.

Suspect this is the cause of #2371.

@BaronGreenback BaronGreenback marked this pull request as draft May 2, 2021
@BaronGreenback BaronGreenback force-pushed the BaronGreenback:AnotherDlnaFix branch from 22be473 to 92d9358 May 2, 2021
@BaronGreenback BaronGreenback marked this pull request as ready for review May 2, 2021
@BaronGreenback BaronGreenback changed the title DLNA fix : Make media visible DLNA fix : Make media visible and playable May 2, 2021
BaronGreenback added a commit to BaronGreenback/jellyfin that referenced this pull request May 2, 2021
Co-authored-by: Cody Robibero <cody@robibe.ro>
@Ullmie02 Ullmie02 added this to Active PRs in Current Release via automation May 4, 2021
@BaronGreenback
Copy link
Contributor Author

@BaronGreenback BaronGreenback commented May 17, 2021

TV went bang - new one has problems with dlna server. Blocked whilst i make sure this change isn't the cause.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Current Release
  
Active PRs
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants