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

When MultipartBody is serialized the values are generated with CRLF #256

Conversation

MartinM85
Copy link
Contributor

Fix for #247

@MartinM85 MartinM85 requested a review from a team as a code owner June 26, 2024 06:30
@andrueastman
Copy link
Member

Any chance we can clarify things out in the issue at #247 first?
I believe there is a possibility that no action is needed here..

@MartinM85
Copy link
Contributor Author

@andrueastman You are right, with text/plain everything works fine in the latest version. Not sure why I was using application/x-www-form-urlencoded

@andrueastman
Copy link
Member

@MartinM85 Thanks for confirming. I think we can close this one for now.

If it makes sense, the next action steps would be looking into whether we need to make a change in the application/x-www-form-urlencoded serialization writer as highlighted in #247

@andrueastman
Copy link
Member

Closing this one for now to follow up on #247

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