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

S3 Download manager should handle no Content Range and Length for first chunk #419

Closed
jasdel opened this issue Nov 3, 2015 · 0 comments
Closed
Assignees
Labels
bug This issue is a bug.

Comments

@jasdel
Copy link
Contributor

jasdel commented Nov 3, 2015

Right now for the first chunk downloader's uses setTotalBytes to determine if it should sleep while waiting for the first chunk's response before requesting the subsequent chunks concurrently. If the total bytes are not updated by the setTotalBytes the downloader.download() the first chunk will never be seen as completed. In this case the download manager should fallback to disabling its concurrency and download the object one chunk at a time since it doesn't know the total size to divide the object into.

jasdel referenced this issue Nov 3, 2015
Fixes a bug when the S3 object being download is not chunked and the
full contents are provided in the response. Content-Range header will
not be provided, and S3 Download manager needs to fallback to
Content-Length instead to get the total length of the object.

Fix # 417
@jasdel jasdel added the bug This issue is a bug. label Nov 3, 2015
xibz pushed a commit to xibz/aws-sdk-go that referenced this issue Jan 15, 2016
@xibz xibz closed this as completed in #510 Jan 18, 2016
xibz added a commit that referenced this issue Jan 18, 2016
jasdel added a commit that referenced this issue Jan 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug.
Projects
None yet
Development

No branches or pull requests

2 participants