[rust-server] Drop `file` support (!547) · Merge requests · OpenAPI Tools / openapi-generator

Merged Administrator requested to merge github/fork/bjgill/drop-file-support into master Jul 12, 2018

Created by: bjgill

PR checklist

Read the contribution guidelines.
Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{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\.
Filed the PR against the correct branch: master, 3.1.x, 4.0.x. Default: master.
Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

In swagger v2, we had 'binary', 'byte', and 'file'. OpenAPI v3 only has the former two. This commit drops the old 'file' handling. This has the side-effect of removing a half-complete implementation of form parameter handling.

This removes the ability to send files as streams, so will make life harder for those wishing to send large files without running out of memory.

See https://github.com/OpenAPITools/openapi-generator/issues/307 and https://github.com/OpenAPITools/openapi-generator/issues/538 for the motivation behind this PR.