Skip to content

[Python] better body parameter support for other "text" formats such as xml, yaml, plain#3726

Merged
wing328 merged 3 commits intoswagger-api:masterfrom
bkryza:bugfix/issue-3723
Oct 2, 2016
Merged

[Python] better body parameter support for other "text" formats such as xml, yaml, plain#3726
wing328 merged 3 commits intoswagger-api:masterfrom
bkryza:bugfix/issue-3723

Conversation

@bkryza
Copy link
Copy Markdown
Contributor

@bkryza bkryza commented Sep 6, 2016

PR checklist

  • Read the contribution guildelines.
  • Ran the shell/batch script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates)
  • Filed the PR against the correct branch: master for non-breaking changes and 2.3.0 branch for breaking (non-backward compatible) changes.

Description of the PR

Fix for the #3723.

The logic in rest.py for generating the HTTP request from provided arguments now accepts body arguments of string type regardless of declared content-type.

@wing328 wing328 merged commit 40a1879 into swagger-api:master Oct 2, 2016
acramatte added a commit to comerge/swagger-codegen that referenced this pull request Oct 4, 2016
* upstream/master: (79 commits)
  add undertow
  Add a new cli command to output version information (2nd attempt) swagger-api#3892 (swagger-api#3899)
  fix python flask controller without tag (default_controller)
  [aspnet5] Fix basePath application to operations (swagger-api#3911)
  Bugfix/issue 3723 (swagger-api#3726)
  Cgardens nested object regex (swagger-api#3879)
  [Cpprest] Fixing issue swagger-api#3773 (swagger-api#3876)
  escape callback parameter for java(okhttp) and python
  fix warning in html generator
  [PHP] fix PHPUnit invocation, add basic phpunit.xml.dist (swagger-api#3864)
  [Java] Remove duplicated model description in Spring, JAX-RS models (swagger-api#3887)
  [PHP] Better PSR2 compatibility (swagger-api#3863)
  Mention security script in pull request template
  [Swift] Use thread safe manager dictionary
  Replace ^M with new line (\r) in mustache template (swagger-api#3865)
  [swfit] fix url param with base name
  [JaxRS]Show correct default value on CLI option description (swagger-api#3862)
  add title, description to HTML output (swagger-api#3860)
  fix trailing comma in go api client
  fix typescript-fetch base path by removing ending slash
  ...
@wing328 wing328 changed the title Bugfix/issue 3723 [Python] better body parameter support for other "text" formats such as xml, yaml, plain Feb 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants