-
-
Notifications
You must be signed in to change notification settings - Fork 779
Fix for aiohttp and multipart/form-data uploads #1222
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
b131e21
Added unit tests to demonstrate the problems of https://github.com/za…
ddurham2 2af3c48
now splitting out multipart POSTs into files[] and form[], handling d…
ddurham2 6ba7927
rewrote how operations/openapi.py::_get_body_argument() works to bett…
ddurham2 c277980
Adding unit tests to improve code converage test
ddurham2 41e03cd
Merge remote-tracking branch 'upstream/master' into fix_975-renewed
ddurham2 5b69e90
post merge fixes - using 'async' keyword now in new unit test file
ddurham2 e214f45
unit test improvements -- now testing the contents of the files we up…
ddurham2 1628d36
making some code a bit clearer regarding duplicate names of file subm…
ddurham2 13488a5
Merge branch 'master' of https://github.com/zalando/connexion into fi…
ddurham2 c6c5009
Merge branch 'main' of https://github.com/spec-first/connexion into f…
ddurham2 d4da76c
fixing up unit tests since merging main
ddurham2 b0bb367
fixing isort-check-tests and flake8
ddurham2 41ebb71
clarified a comment
ddurham2 cbaff89
comment correction
ddurham2 83c8b4d
after discussions with maintainer, reverted _get_body_argument back t…
ddurham2 40a65f7
fixing unit tests after after reverting _get_body_argument behavior
ddurham2 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Merge branch 'main' of https://github.com/spec-first/connexion into f…
…ix_975-renewed
- Loading branch information
commit c6c5009257718f1b25a846f344e4bdd660ea8a87
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
You are viewing a condensed version of this merge commit. You can view the full changes here.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.