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

Add DLNA headers #242

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

Add DLNA headers #242

wants to merge 2 commits into from

Conversation

luigiplr
Copy link

Add relevant DLNA headers to the server.

if a subtitle by the name of video.srt is present in the same folder as
the video file => use it.
@luigiplr luigiplr changed the title Add DLNA headers / subtitle header (s) Add DLNA headers Nov 10, 2015
@mafintosh
Copy link
Owner

could you remove the 2nd pair of DLNA headers we set further down then? That way we don't set them twice. Other than that LGTM

@jaruba
Copy link

jaruba commented Apr 15, 2016

Well, the second pair of DLNA headers are the entire commit as he removed the subtitle stuff.

But he removed the subtitle stuff because he realized that he's setting a subtitle from a server that he wasn't creating.

Even if he would though.. Or tweak it to use peerflix's http server for this, setting subtitles with the CaptionInfo.sec header works only for Samsung devices, nothing else.

@@ -164,6 +164,8 @@ var createServer = function (e, opts) {
response.statusCode = 206
response.setHeader('Content-Length', range.end - range.start + 1)
response.setHeader('Content-Range', 'bytes ' + range.start + '-' + range.end + '/' + file.length)
response.setHeader('transferMode.dlna.org', 'Streaming')
response.setHeader('contentFeatures.dlna.org', 'DLNA.ORG_OP=01;DLNA.ORG_CI=0;DLNA.ORG_FLAGS=017000 00000000000000000000000000')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's already done above fyi

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oups sorry reviving an old thread mb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants