Add a segmentRequestHeaders option for DASH segment requests - #1881
Add a segmentRequestHeaders option for DASH segment requests#1881TomaszOszczepalski wants to merge 2 commits into
Conversation
|
Hi, Thanks for the proposal, I agree that this is a real use case, yet in that scenario I would prefer relying on the It's true we don't call it for low-latency content, I think ideally we should focus on that part and update the API to also answer your usage. Why low-latency contents do not trigger the
|
|
Hi, Thanks for the detailed explanation. I agree that keeping I see For our authorization use case, From the RxPlayer API perspective, I think it would be important that:
An I can adapt our bridge to the resulting API, so my main concern is finding a solution that remains coherent and robust for RxPlayer itself. From that perspective, extending |
Yes, I just think that if low-latency is a goal here, we have to also call the
Of course. In our case the RxPlayer knows how to detect decodable subparts so the application could just be streaming the chunk data as it becomes available to it. |
|
I've attempted something: #1883 For your case it would be doing something like: |
Why
Some CDNs require an authorization header on audio and video segment
requests, while the Manifest request should remain unchanged.
A custom
segmentLoadercannot handle this in low-latency mode becauseRxPlayer intentionally uses its internal chunked
fetchimplementation whenlowLatencyModeis enabled.What changed
This adds an optional
segmentRequestHeadersproperty toloadVideo.The configured headers are merged into built-in DASH requests for:
The headers are not added to:
RxPlayer's internally generated CMCD and
Rangeheaders remain authoritative.The option is optional, so existing behavior is unchanged when it is absent.
Example