-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[Rust Server] Support multipart/form_data request bodies #2846
[Rust Server] Support multipart/form_data request bodies #2846
Conversation
- Support multipart/form_data in the Rust Server - Add a new test API to test the change. - Update the examples to match
CI Failure is in the elm code, which should be unrelated (e.g. it's seen in the merge of #2112). |
@richardwhiuk right, the CI failure (addressed in the latest master) has nothing to do with this PR. If you are you free, let's have a quick chat via https://gitter.im (ID: wing328). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR - looks good. I've just got a dependency version question and a minor style tweak.
PR checklist
./bin/
to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.sh
,./bin/openapi3/{LANG}-petstore.sh
if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in.\bin\windows\
. If contributing template-only or documentation-only changes which will change sample output, be sure to build the project first.master
,. Default:3.4.x
,4.0.x
master
.@frol, @farcaller, @bjgill
Description of the PR
This has been written, and reviewed by myself, and others at @Metaswitch.