#1442 Implement Fetch Stream API

Open
opened 1 year ago by wolfbeast · 2 comments
wolfbeast commented 1 year ago (Migrated from github.com)

We should probably aim for a self-hosted JS implementation here for the sake of maintainability and to reduce risk of C++ sec issues. Might not be as performant but shouldn’t be an issue for normal use.

We should probably aim for a self-hosted JS implementation here for the sake of maintainability and to reduce risk of C++ sec issues. Might not be as performant but shouldn't be an issue for normal use.
Moonchild self-assigned this 6 months ago
Moonchild commented 6 months ago
Owner

On second thought interacting with the DOM is very complicated doing this as self-hosted JS, so it’s probably better to follow Mozilla’s implementation.
I have an almost complete implementation in JS that works for a number of tests (basic functionality works, but ReadableStream combo with Fetch is still a problem) however it’s not actually giving me the plumbing I need to further integrate this with the DOM and adding writeable streams etc. -- It’s also 140+ kB of js which really isn’t going to do anyone any good.

So I’ll be backing that out and looking at the C++ version instead, but wanted to keep it in history at least in case we do want to keep stuff from it in the future/use it as a template.

On second thought interacting with the DOM is very complicated doing this as self-hosted JS, so it's probably better to follow Mozilla's implementation. I have an almost complete implementation in JS that works for a number of tests (basic functionality works, but ReadableStream combo with Fetch is still a problem) however it's not actually giving me the plumbing I need to further integrate this with the DOM and adding writeable streams etc. -- It's also 140+ kB of js which really isn't going to do anyone any good. So I'll be backing that out and looking at the C++ version instead, but wanted to keep it in history at least in case we do want to keep stuff from it in the future/use it as a template.
Moonchild removed their assignment 6 months ago
Moonchild added the
Bounty
label 6 months ago
Moonchild added the
On Hold
label 6 months ago
Moonchild commented 6 months ago
Owner

This is now officially on hold because I’m unable to follow Mozilla’s snake river.
If anyone thinks they can, in fact, do this, they can take it -- I’ll leave the work branch available for that purpose since it does have functional code in it.

This is now officially on hold because I'm unable to follow Mozilla's snake river. If anyone thinks they can, in fact, do this, they can take it -- I'll leave the work branch available for that purpose since it does have functional code in it.
Moonchild removed this from the 29.1.0 milestone 4 months ago
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.