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

Only one header value of several is sent with MultipartForm body #2704

Merged
merged 2 commits into from
Jan 20, 2025

Conversation

tsegismont
Copy link
Contributor

See #2678

ilia1243 and others added 2 commits January 20, 2025 17:39
We should keep and simplify multipart form headers copying.
The changes in HttpRequestImpl can be reverted.

Signed-off-by: Thomas Segismont <[email protected]>
@tsegismont tsegismont merged commit 29177db into vert-x3:master Jan 20, 2025
7 checks passed
@tsegismont tsegismont deleted the ilia1243-several-header-values branch January 20, 2025 16:56
tsegismont added a commit that referenced this pull request Jan 20, 2025
* [2678] Send multiple headers for url-encoded and multipart-form bodies

Bug: #2678

* Updates after review

We should keep and simplify multipart form headers copying.
The changes in HttpRequestImpl can be reverted.

Signed-off-by: Thomas Segismont <[email protected]>

---------

Signed-off-by: Thomas Segismont <[email protected]>
Co-authored-by: ilia1243 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants