-
-
Notifications
You must be signed in to change notification settings - Fork 772
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.
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
Fix RuntimeError exception from **application/x-www-form-urlencoded** (see also PR# 1222) #1348
Fix RuntimeError exception from **application/x-www-form-urlencoded** (see also PR# 1222) #1348
Conversation
- Taken mostly from existing PR: spec-first#987
…uplicate keys as the rest of connexion expects - Based parly on existing PR: spec-first#987
…er build the arguments[] list according to what the spec says and what the handler accepts. This fixes a bug when requests contain mixed files and form values and the handler is expecting variable names matching the request property names.
I'm getting an odd RuntimeError with this patch.
Alas, although my application seems to be returning a byte-based request body, decoding it as UTF-8 in my application doesn't seem to work. I'm unsure about the specific source of this error. If anyone has an idea, please let me know. Thus, I've converted this PR to 'draft' pending arriving at a better understanding as to what is going on. |
…b.com/STARInformatics/connexion into fix-urlencoded-body-parameter-parsing
Closing due to inactivity. |
Adds to PR #1222 to fix a RuntimeError exception from application/x-www-form-urlencoded processes here, as well as the multipart/form-data issues already addressed by the PR.
Changes proposed in this pull request:
update
method (next line). Parsing the byte string out creates a dictionary which is better behaved (coded a local method,parse_body_parameters(self, body)
, to achieve this).