Skip to content

Conversation

padamstx
Copy link
Contributor

@padamstx padamstx commented Jan 6, 2023

This commit modifies the RequestBuilder.Build() method's handling of multi-part forms so that the request body is created in a streamed fashion, rather than making an entire copy of each mime part's contents.
This change will be especially useful when a multi-part form contains one or more parts containing large files.

Signed-off-by: Phil Adams phil_adams@us.ibm.com

@padamstx padamstx self-assigned this Jan 6, 2023
@padamstx padamstx requested review from dpopp07 and pyrooka January 6, 2023 23:34
Copy link
Member

@pyrooka pyrooka left a comment

Choose a reason for hiding this comment

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

LGTM!

Base automatically changed from fix-vpcauth to main January 9, 2023 15:07
Copy link
Contributor

@dpopp07 dpopp07 left a comment

Choose a reason for hiding this comment

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

Looks good to me! 👍

This commit modifies the RequestBuilder.Build() method's
handling of multi-part forms so that the request body
is created in a streamed fashion, rather than making
an entire copy of each mime part's contents.
This change will be especially useful when a multi-part
form contains one or more parts containing large files.

Signed-off-by: Phil Adams <phil_adams@us.ibm.com>
@padamstx padamstx merged commit 7df8c71 into main Jan 9, 2023
@padamstx padamstx deleted the form-streaming branch January 9, 2023 23:06
ibm-devx-sdk pushed a commit that referenced this pull request Jan 9, 2023
# [5.11.0](v5.10.3...v5.11.0) (2023-01-09)

### Features

* create multi-part form as a streamed request body ([#169](#169)) ([7df8c71](7df8c71))
@ibm-devx-sdk
Copy link

🎉 This PR is included in version 5.11.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants