Skip to content

Add support for wrapping arbitrary request bodies#81

Merged
Pagten merged 5 commits into
masterfrom
pieagt/add-support-for-wrapping-arbitrary-bodies
Jun 22, 2026
Merged

Add support for wrapping arbitrary request bodies#81
Pagten merged 5 commits into
masterfrom
pieagt/add-support-for-wrapping-arbitrary-bodies

Conversation

@Pagten

@Pagten Pagten commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

This PR refactors shared_body::SharedBody into a new body::RequestBody type that not only supports shared bodies constructed from an Arc<Vec<u8>> or &'static [u8] (like the original SharedBody) but can also wrap arbitrary other hyper::body::Body implementations.

@Pagten Pagten force-pushed the pieagt/add-support-for-wrapping-arbitrary-bodies branch from 5e523a0 to 99fa98f Compare June 21, 2026 20:09
@Pagten Pagten force-pushed the pieagt/add-support-for-wrapping-arbitrary-bodies branch from 99fa98f to f0c10e6 Compare June 21, 2026 20:09
heilhead
heilhead previously approved these changes Jun 22, 2026

@heilhead heilhead left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good!

adrian-brabete
adrian-brabete previously approved these changes Jun 22, 2026
@Pagten Pagten dismissed stale reviews from adrian-brabete and heilhead via d5dd440 June 22, 2026 08:17
Comment thread simple-hyper-client/src/error.rs Outdated
heilhead
heilhead previously approved these changes Jun 22, 2026
adrian-brabete
adrian-brabete previously approved these changes Jun 22, 2026
@Pagten Pagten dismissed stale reviews from adrian-brabete and heilhead via d7fc726 June 22, 2026 08:46
@Pagten Pagten force-pushed the pieagt/add-support-for-wrapping-arbitrary-bodies branch from d5dd440 to d7fc726 Compare June 22, 2026 08:46
@Pagten Pagten merged commit d711be9 into master Jun 22, 2026
6 checks passed
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.

3 participants