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

Differentiate playlist/segment http downloading #602

Closed
shaharmor opened this issue Nov 20, 2016 · 3 comments
Closed

Differentiate playlist/segment http downloading #602

shaharmor opened this issue Nov 20, 2016 · 3 comments
Assignees
Labels
flag: good first issue This might be a relatively easy issue; good for new contributors status: archived Archived and locked; will not be updated type: enhancement New feature or request
Milestone

Comments

@shaharmor
Copy link

  • What version of Shaka Player are you using? latest

When overriding the httpPlugin, there is no easy way from within the plugin itself to know if the request is a playlist or a segment request.
It may be required to have different logic based on that info.

The two options are either to pass that info as a parameter to the plugin itself, or have two separate plugins (Which by default will use the same module), and then we will be able to override one or the other.

@joeyparrish
Copy link
Member

The two parameters to a networking plugin are uri and request. We could easily add RequestType to the Request structure.

Then you could compare request.type to RequestType.SEGMENT or RequestType.MANIFEST in your plugin. Would that help?

@joeyparrish joeyparrish added the type: enhancement New feature or request label Nov 29, 2016
@joeyparrish joeyparrish added this to the v2.1.0 milestone Nov 29, 2016
@shaharmor
Copy link
Author

shaharmor commented Nov 29, 2016 via email

@joeyparrish joeyparrish added the flag: good first issue This might be a relatively easy issue; good for new contributors label Dec 19, 2016
@joeyparrish
Copy link
Member

We're going to pass an additional argument to the plugins instead of adding type to the request structure.

@shaka-project shaka-project locked and limited conversation to collaborators Mar 22, 2018
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Apr 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
flag: good first issue This might be a relatively easy issue; good for new contributors status: archived Archived and locked; will not be updated type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants