feat: add support for browser streaming#721
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
This CI is failing because my update to Update: This issue is mentioned in #671 |
a81e50a to
021bf27
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
8e5a85d to
b1cde9b
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
b8e6da5 to
d529151
Compare
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
After fixing the CodeBuild image, the CI finally succeeds. Now this is ready to be reviewed. |
3d60c59 to
9e81bd4
Compare
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread. |
Reopen #581
Per #102 browser streaming as removed from preview because
ReadableStreamsupport in browsers was not ideal back then. Now since most of the browser vendors support ReadableStream, we can enable the feature.Previously, the
fetch-http-handlerreturns a promise ofblobfor body, even when we expect the body to be streaming sometimes(e.g. S3 GetObject). After these change, we can return aReadableStreamfor streaming response in browser. This would avoid loading all the resposne body data in the brwoser memory.reference:
ReadableStreambrowser support: https://caniuse.com/#search=ReadableStreamReadableStreamupdate for browser support data: Update ReadableStream compatibility in Safari mdn/browser-compat-data#5424Responseconstructor: https://fetch.spec.whatwg.org/#response-classResponseconstructor browser compatibility: https://caniuse.com/#feat=mdn-api_response_responseBody(e.g.Body.arraybuffer()) update for browser support data: add body api safari, safari mobile, chrome android support mdn/browser-compat-data#5490BodyandReadablestream support data is not accurate. They are actually supported bycurrent mainstream browsers, desktop and mobile. See above 2 PRs for more information.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.