Created by: euank
PR checklist
-
Read the contribution guidelines. -
Ran the shell script under ./bin/to update Petstore sample so that CIs can verify the change. -
Copied the technical committee
cc @frol @farcaller @bjgill
Description of the PR
This PR contains two changes which I needed to make for a particular swagger client to work.
The first moves most request logic into a standalone file / method. This makes it much erasier to avoid variable conflicts since the Request.execute method doesn't have the generated params in its scope, and as such doesn't have to worry about conflicting with them.
I intend for .execute to behave identically to what we previously generated per operation.
I did, however, also implement form data while I was at it. That is included as a second separate commit, and I can file it separately if that's preferred.
Fixes #512 (closed), #525 (closed)