Skip to content

Add Form::set_boundary to customize multipart boundary#2814

Open
LucasPickering wants to merge 1 commit intoseanmonstar:masterfrom
LucasPickering:2374-custom-boundary
Open

Add Form::set_boundary to customize multipart boundary#2814
LucasPickering wants to merge 1 commit intoseanmonstar:masterfrom
LucasPickering:2374-custom-boundary

Conversation

@LucasPickering
Copy link
Copy Markdown
Contributor

@LucasPickering LucasPickering commented Sep 4, 2025

Customizing the boundary makes it easier to write tests that assert on the Content-Type header and body, as the content can be static. Method documentation sufficiently describes the risk of doing this.

Closes #2374

LucasPickering added a commit to LucasPickering/slumber that referenced this pull request Sep 26, 2025
LucasPickering added a commit to LucasPickering/slumber that referenced this pull request Sep 26, 2025
@LucasPickering
Copy link
Copy Markdown
Contributor Author

@seanmonstar Sorry to ping you. Any chance you could review this? Thanks!

Customizing the boundary makes it easier to write tests that assert on the Content-Type header and body, as the content can be static. Method documentation sufficiently describes the risk of doing this. I added an integration test for the async use case. The blocking use case is so similar that I didn't think it deserved its own test.

Closes seanmonstar#2374
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.

[Feature Request] Add option to customize boundary in multipart request

1 participant