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
fix: Determine Content-Length
Before Attempting Multi-chunk Upload
#2074
Conversation
Content-Length
Before Attempting Multi-chunk Upload
…pis/nodejs-storage into fix-divisible-chunk-size
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discussed the implementation in detail with @danielbankhead.
The PR LGTM, it introduces not only unit tests but also system tests which validate critical multi chunk use cases which were not being tested before against the live service.
Complexity of resumable media API with streams caused the following use cases to not be captured: aa80374
The tests that failed in this commit now pass as expected.
Fixes #2073 🦕