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

feat(xhr-http-handler): add XMLHttpRequest http handler to use with lib-storage Upload #3798

Merged
merged 13 commits into from
Jul 15, 2022

Conversation

kuhe
Copy link
Contributor

@kuhe kuhe commented Jul 13, 2022

Issue

Finer upload progress grain (#3101)

Description

Adds an XMLHttpRequest http handler. This can be used instead of the fetch handler in certain cases.
Specifically, it gives fine-grained progress updates for single part and multi-part uploads via lib-storage'sUpload.

Testing

Unit tests and manual browser testing with Upload from lib-storage.

Additional context

This has only been tested against S3 in browsers.

@kuhe kuhe added the pr/needs-review This PR needs a review from a Member. label Jul 13, 2022
@kuhe kuhe requested a review from a team as a code owner July 13, 2022 19:26
@kuhe kuhe self-assigned this Jul 13, 2022
@AllanZhengYP AllanZhengYP self-requested a review July 13, 2022 23:42
Copy link
Contributor

@AllanZhengYP AllanZhengYP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Only with a nit.

packages/xhr-http-handler/README.md Show resolved Hide resolved
@kuhe kuhe merged commit 7462b07 into aws:main Jul 15, 2022
@kuhe kuhe deleted the xhr branch July 15, 2022 17:26
@github-actions
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr/needs-review This PR needs a review from a Member.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants