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

Correct instantiation of HttpVideoSource with no content protection. #292

Merged
merged 1 commit into from
Feb 27, 2016

Conversation

sanbornhilland
Copy link
Contributor

Passing an empty config object is no longer acceptable because the
keySystem property is mandatory on drmConfig objects.

Also modified HttpVideoSource constructor comment so that the drmConfig
comments are the single source of truth regarding well-formed drmConfig
objects.

Passing an empty config object is no longer acceptable because the
keySystem property is mandatory on drmConfig objects.

Also modified HttpVideoSource constructor comment so that the drmConfig
comments are the single source of truth regarding well-formed drmConfig
objects.
@joeyparrish joeyparrish self-assigned this Feb 26, 2016
@joeyparrish
Copy link
Member

Looks good, but be warned: HttpVideoSource was originally used for testing purposes. We've deprecated it and no equivalent is planned for v2.

joeyparrish added a commit that referenced this pull request Feb 27, 2016
Correct instantiation of HttpVideoSource with no content protection.
@joeyparrish joeyparrish merged commit 96bf5b2 into shaka-project:master Feb 27, 2016
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Jul 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants