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

Files that are exactly the chunk size can be uploaded as is #287

Merged
merged 1 commit into from
Feb 7, 2014

Conversation

girving
Copy link
Contributor

@girving girving commented Feb 7, 2014

Previously, a file was uploaded as a single chunk if

fsize < self.max_file_size

This is wrong: files that are exactly the max_file_size can also be
uploaded as a single chunk. The fix is to replace < with <=:

fsize <= self.max_file_size

This bug is problematic when attempting to replicate pyrax's normal
chunk uploading behavior, while still using pyrax to upload chunks.

This is a resubmission of #270.

Previously, a file was uploaded as a single chunk if

  fsize < self.max_file_size

This is wrong: files that are exactly the max_file_size can also be
uploaded as a single chunk.  The fix is to replace < with <=:

  fsize < self.max_file_size

This bug is problematic when attempting to replicate pyrax's normal
chunk uploading behavior, while still using pyrax to upload chunks.
EdLeafe added a commit that referenced this pull request Feb 7, 2014
Files that are exactly the chunk size can be uploaded as is
@EdLeafe EdLeafe merged commit 13893df into pycontribs:working Feb 7, 2014
@girving girving deleted the fix-chunk-size-upload branch March 11, 2014 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants