[Feature Request] Pass mpd URL custom headers to actual video files?

[matthuisman]

I have a .strm file like this:

#KODIPROP:inputstreamaddon=inputstream.adaptive
#KODIPROP:inputstream.adaptive.manifest_type=mpd
#KODIPROP:inputstream.adaptive.license_type=com.widevine.alpha
#KODIPROP:inputstream.adaptive.license_key=https://wvlic.brightcove.com/proxy/5375730767001|Content-Type=application%2Fx-www-form-urlencoded&X-Forwarded-For=202.89.4.222|A{SSM}|
http://dashtvnz-a.akamaihd.net/963482467001/963482467001_5375755467001_5375730767001.mpd?pubId=963482467001&videoId=5375730767001|X-Forwarded-For=202.89.4.222

For a GEO blocked service.

Here's the log:
https://pastebin.com/raw/T8jsHLXV

You can see it get's the license all OK etc.
However, it fails to open http://dashtvnz-a.akamaihd.net/963482467001/5375755266001/video/1/init.mp4

This will be due to CURL not setting the X-Forwarded-For (it only sets keep alive)

Would it be possible to update the code to pass on any custom stream headers to the actual video files?

I guess somewhere around here it would happen:
https://github.com/peak3d/inputstream.adaptive/blob/master/src/main.cpp#L217

[matthuisman]

Possibly it could be an extra Addon Setting if that's easier to implement?
"Extra Headers" or something like that?
Then that download function (lines 189 & 214) just appends |{headers} to the end of the URL string?

xbmc->CURLCreate(url + "|" + xbmc->GetSetting("EXTRAHEADERS", (char*)&buf));

Pretty sure the CurlCreate takes care of splitting the string on | and parsing the headers?

[matthuisman]

The downside to above is that add-ons like the one I write won't be able to set the extra headers.

[glennguy]

@matthuisman It seems to be able to do this already, I think you need to URL encode the string properly - '&' should be '%2F'. i.e

#KODIPROP:inputstream.adaptive.license_key=https://wvlic.brightcove.com/proxy/5375730767001|Content-Type=application%2Fx-www-form-urlencoded%2FX-Forwarded-For=202.89.4.222|A{SSM}|

[matthuisman]

@glennguy

That's not quite the issue (the license key actually works fine)

The issue is that the X-Forwarded-For at the end of the MPD URL is not passed onto the actual video (init.mp4 / seg-1.m4f etc etc) files.

1. The code fetches the MPD file correctly using the X-Forward
2. The code get's the actual video files from the MPD file (eg. init.mp4)
3. The code does not add X-Forward when requesting these video files

If you see my log here:
https://pastebin.com/B8L3gYK8

You can see CURL adding the custom header for the MPD & key URLS.
However, it doesn't add the custom header for any further files (init.mp4 etc) - it only adds a "keep-alive" header.

So I thought the code could store the custom headers in a variable when first fetching the MPD file.
Then these headers are just appended to any further URLS.

If it worked correctly, you should be able to play that STRM from my first post even though it's region locked to New Zealand :)

[glennguy]

@matthuisman my bad, I did a pretty bad job of reading your first post.

I've made a fix that pretty much follows your suggestion of storing the headers in a variable and append them to segment downloads. It works for your strm file. It's an easy but probably not ideal implementation. I'll make a PR.

[matthuisman]

@glennguy
You are a LEGEND!

You could also append the X-Forward headers in your Aussie add-ons so us Kiwis can use them :)

[matthuisman]

Going to close this issue and continue convo on pull request :)