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

Add some api for web readable streams #907

Closed
jimmywarting opened this issue Sep 10, 2016 · 4 comments
Closed

Add some api for web readable streams #907

jimmywarting opened this issue Sep 10, 2016 · 4 comments
Labels

Comments

@jimmywarting
Copy link
Contributor

@jimmywarting jimmywarting commented Sep 10, 2016

Given that it's all about using torrent in the web i think it would make since to support the standard web stream also.

Not saying we should go and replace the hole node stream and go with web streams instead.
But for a way to get it as a instance of ReadableStream from the api so that it can be consumed by the new FileSystem api, fetch upcoming streaming upload, to streamsaver and all new stuff that i don't know about yet

@feross

This comment has been minimized.

Copy link
Member

@feross feross commented Oct 5, 2016

It sounds like a great idea to figure out how to turn node.js streams into W3C streams. I'm surprised there isn't a module for this already. Want to go write it? 😉

I don't think this belongs in the library itself, though. Node.js streams work well and they're by far the most widely used stream implementation. I'm happy to reconsider this if W3C streams get a ton of adoption.

@feross feross added the enhancement label Oct 5, 2016
@feross feross closed this Oct 5, 2016
@josephfrazier

This comment has been minimized.

Copy link
Member

@josephfrazier josephfrazier commented Oct 7, 2016

It looks like node-web-streams has some conversion functions: https://github.com/gwicke/node-web-streams

@jimmywarting

This comment has been minimized.

Copy link
Contributor Author

@jimmywarting jimmywarting commented Oct 7, 2016

hmm, interesting. Maybe wish he rather built a conversion api instead of depending on web-stream-polyfill

@lock

This comment has been minimized.

Copy link

@lock lock bot commented May 4, 2018

This thread has been automatically locked because it has not had recent activity. To discuss futher, please open a new issue.

@lock lock bot locked as resolved and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.