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

[Request] - Support Range header for streaming media to Safari #806

Closed
alexbowers opened this issue Nov 1, 2018 · 2 comments
Closed

[Request] - Support Range header for streaming media to Safari #806

alexbowers opened this issue Nov 1, 2018 · 2 comments
Labels
accepted An accepted request or suggestion request Request for new functionality

Comments

@alexbowers
Copy link
Contributor

  1. Why you believe this feature is necessary.

Currently you can use Rocket to serve static assets such as Images, JS files, etc easily and it works, however if you try to serve video media (and potentially audio, haven't tested audio), it will only work in some browsers, because of browser support and leniency on the Range and RangeByte headers being respected and the correct information being returned.

Currently, it seems to send all bytes at the same time, even if the Range requested was only bytes 0-1 (which is the header request made by Safari to determine media type).

If you return all content back on that request, Chrome and other browsers are happy to play that content and ignore the mistake, however Safari refuses to play the content.

  1. A convincing use-case for this feature.

Using Rocket to serve static movie assets as part of your website is not currently possible, it currently requires either third-party (S3) or alternate server software for serving that asset, which isn't desirable if you want to lock the content behind auth walls etc in your application.

  1. Why this feature can't or shouldn't exist outside of Rocket.

It can exist outside of Rocket, however it makes using Rocket as a static server unusable since a common type of asset is not able to be sent to a popular browser (Mobile safari + Desktop).

@SergioBenitez SergioBenitez added request Request for new functionality accepted An accepted request or suggestion labels Nov 1, 2018
@alexbowers
Copy link
Contributor Author

Hi @SergioBenitez congrats on the 0.4.0 release.

Can i request that this feature be added as part of the 0.5.0 release?

@SergioBenitez
Copy link
Member

Closing in favor of #95.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted An accepted request or suggestion request Request for new functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants