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

Expose a Form::into_reader() method on blocking multipart forms. #2524

Merged

Conversation

obi1kenobi
Copy link
Contributor

An example use case is compressing multipart form data with zstd. The entire contents of the payload have to be compressed together, which requires accessing the contents of the Form.

The existing reader functionality already implements what we need, but just isn't publicly accessible. This PR adds a pub method for it.

An example use case is compressing multipart form data with zstd. The
entire contents of the payload have to be compressed together, which
requires accessing the contents of the `Form`.

The existing reader functionality already implements what we need, but
just isn't publicly accessible. This PR adds a `pub` method for it.
obi1kenobi added a commit to obi1kenobi/reqwest that referenced this pull request Jan 9, 2025
The async equivalent of the change in seanmonstar#2524.

An example use case is compressing multipart form data with zstd.
The entire contents of the payload have to be compressed together,
which requires accessing the contents of the Form.

The existing stream functionality mostly implements what we need,
but just isn't publicly accessible. This PR adds a pub method for it.
obi1kenobi added a commit to obi1kenobi/reqwest that referenced this pull request Jan 9, 2025
The async equivalent of the change in seanmonstar#2524.

An example use case is compressing multipart form data with zstd.
The entire contents of the payload have to be compressed together,
which requires accessing the contents of the Form.

The existing stream functionality mostly implements what we need,
but just isn't publicly accessible. This PR adds a pub method for it.
@seanmonstar seanmonstar merged commit f7d929f into seanmonstar:master Jan 9, 2025
36 checks passed
@obi1kenobi obi1kenobi deleted the pg/blocking_multipart_into_reader branch January 9, 2025 21:07
seanmonstar pushed a commit that referenced this pull request Jan 9, 2025
…2525)

The async equivalent of the change in #2524.

An example use case is compressing multipart form data with zstd.
The entire contents of the payload have to be compressed together,
which requires accessing the contents of the Form.

The existing stream functionality mostly implements what we need,
but just isn't publicly accessible. This PR adds a pub method for it.
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