Add updateContentUrls API
#1182
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
peaBerberian
force-pushed
the
feat/updatecontenturls
branch
3 times, most recently
from
2fdfe56
to
703195c
Compare
peaBerberian
force-pushed
the
feat/updatecontenturls
branch
from
703195c
to
b13c40f
Compare
This commit adds the `updateContentUrls` API allowing to update at any time and during playback, the URL through which a content's Manifest is reached. This may be used anytime an application prefer a new Manifest URLs to be used, whether it is because the previous URL will soon expire, because the new URL is seen as more qualitative or to rebalance load. The first argument given to `updateContentUrls` is actually an array of strings, listing new URLs from the most prioritized to the least prioritized. This is not used for now, but should soon be used to define fallback URLs when former one fail to fetch the resource. `updateContentUrls` can be given a second argument named `refreshNow` which can be used to ask the RxPlayer to refresh immediately the Manifest behind the given URL. For now, this API will throw for directfile contents, I'm not sure whether it is pertinent to enable it also in this case.
peaBerberian
force-pushed
the
feat/updatecontenturls
branch
from
b13c40f
to
ec01e19
Compare
peaBerberian
force-pushed
the
feat/updatecontenturls
branch
from
131a7ef
to
ca29e21
Compare
peaBerberian
added a commit
that referenced
this pull request
Dec 21, 2022
Add `updateContentUrls` API
peaBerberian
added a commit
that referenced
this pull request
Jan 31, 2023
Add `updateContentUrls` API
peaBerberian
added a commit
that referenced
this pull request
Feb 8, 2023
Add `updateContentUrls` API
peaBerberian
added a commit
that referenced
this pull request
Feb 9, 2023
Add `updateContentUrls` API
Merged
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit adds the
updateContentUrlsAPI allowing to update at any time and during playback, the URL through which a content's Manifest is reached.This may be used anytime an application prefer a new Manifest URLs to be used, whether it is because the previous URL will soon expire, because the new URL is seen as more qualitative or to rebalance load.
The first argument given to
updateContentUrlsis actually an array of strings, listing new URLs from the most prioritized to the least prioritized. This is not used for now, but should soon be used to define fallback URLs when former one fail to fetch the resource.updateContentUrlscan be given a second argument namedrefreshNowwhich can be used to ask the RxPlayer to refresh immediately the Manifest behind the given URL.For now, this API will throw for directfile contents, I'm not sure whether it is pertinent to enable it also in this case.