Skip to content

Conversation

@Jeff-Meadows
Copy link
Contributor

Currently, requests only supports streaming uploads by chunking them.
This commit utilizes the requests_toolbelt's MultipartEncoder to use
multipart-form uploads while still streaming the data from disk.

Fixes #84.

Currently, requests only supports streaming uploads by chunking them.
This commit utilizes the requests_toolbelt's MultipartEncoder to use
multipart-form uploads while still streaming the data from disk.
@boxcla
Copy link

boxcla commented Oct 30, 2015

Verified that @Jeff-Meadows has signed the CLA. Thanks for the pull request!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

remove this stuff

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Strange that this test worked before...

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 65f351f on box:stream_uploads into 51a8a47 on box:master.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

tell() called several times by MultipartEncoder

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 4985525 on box:stream_uploads into 51a8a47 on box:master.

Jeff-Meadows added a commit that referenced this pull request Nov 5, 2015
@Jeff-Meadows Jeff-Meadows merged commit 42a03fa into master Nov 5, 2015
@Jeff-Meadows Jeff-Meadows deleted the stream_uploads branch November 5, 2015 19:43
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.

4 participants