fix(storage): multipart upload is firing on 0 bytes data - Githubissues

aws-amplify / amplify-js

A declarative JavaScript library for application development using cloud services.

https://docs.amplify.aws/lib/q/platform/js

Apache License 2.0

9.44k stars 2.13k forks source link

fix(storage): multipart upload is firing on 0 bytes data #13927

Closed HuiSF closed 1 month ago

HuiSF commented 1 month ago

Description of changes

Check dataByteLength !== undefined instead of consuming the falsy 0

Issue #, if available

Description of how you validated changes

unit tests

Checklist

[x] PR description included
[x] yarn test passes
[x] Unit Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)

Checklist for repo maintainers

[ ] Verify E2E tests for existing workflows are working as expected or add E2E tests for newly added workflows
[ ] New source file paths included in this PR have been added to CODEOWNERS, if appropriate

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.