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.

Sign up for GitHub

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

Jump to bottom

Fixed missing closing boundary string in encoded MultiPartForm #20

Closed
malcommac opened this issue Mar 12, 2022 · 0 comments
Closed

Fixed missing closing boundary string in encoded MultiPartForm #20

malcommac opened this issue Mar 12, 2022 · 0 comments
Assignees
@malcommac
Labels
bug Something isn't working
Milestone
1.2.0

Comments

@malcommac
Copy link
Collaborator

Bug Report

The Multipart form encoding misses the closing boundary string at the end of the encoded body which is in the form:
"\(crlf)--\(boundaryKey)--\(crlf)"

Q A
BC Break yes
Version 1.x
@malcommac malcommac self-assigned this Mar 12, 2022
@malcommac malcommac added this to the 1.2.0 milestone Mar 12, 2022
@malcommac malcommac added the bug Something isn't working label Mar 12, 2022
malcommac added a commit that referenced this issue Mar 12, 2022
@malcommac
new #20: added multipart() init from MultipartForm instance
1038402
@malcommac malcommac mentioned this issue Mar 12, 2022
Merged
@malcommac malcommac changed the title Multipart encoding error closing the boundary string Missing closing boundary string in encoded multipart form Mar 12, 2022
@malcommac malcommac changed the title Missing closing boundary string in encoded multipart form Fixed missing closing boundary string in encoded multipart form Mar 12, 2022
@malcommac malcommac changed the title Fixed missing closing boundary string in encoded multipart form Fixed missing closing boundary string in encoded MultiPartForm Mar 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@malcommac malcommac

Labels
bug Something isn't working
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
1 participant
@malcommac