[#935] supporting application/json within multipart/form-data request body

[ddurham2]

adding support to automatically decode json when a multipart/form-data request body contains a field with an application/json content-type

Fixes #935.

Changes proposed in this pull request:

Fills in a decode step where there's already a TODO (I think) saying it needs to be done for a variety of types.

[fffan64]

It solves my issue, any idea when it will be merged ?
Thank you :)

[kornicameister]

Would empty string be safer here?

[fffan64]

doesn't change that much for me, it is just the default val

[fffan64]

adding support to automatically decode json when a multipart/form-data request body contains a field with an application/json content-type

Fixes #935.

Changes proposed in this pull request:

Fills in a decode step where there's already a TODO (I think) saying it needs to be done for a variety of types.

I forked your modif because i need it in one of my project quickly, thank you

[zhrkevin]

import json
if encoding.get('contentType', None) == "application/json":
    try:
        form_data[k] = json.loads(form_data[k])
    except Exception:
        pass

How do you feel about this?

[kornicameister]

@zhrkevin sorry for late response. It seems to me that me and connexion will part our ways. I am interested more in aiohttp support which seems to be 2nd class citizen here. That made ma start axion. If you or anyone wants to build typesafe all the way API first framework with async in mind...I will gladly accept any help 🤗

[timonbimon]

is connexion still maintained in general?

[kornicameister]

@timonbimon honestly speaking I don't know. It's very painful to say, but all signs says that active maintenance is not so much of a priority for Zalando at the moment. Well, I am not Zalando employee, just observing the situation.

[hjacobs]

@timonbimon @kornicameister sorry for the late answer: I'm one of the maintainers and currently don't have Connexion much on my radar. We have an internal discussion on how we can get more people involved and help maintain Connexion.

[hjacobs]

In order to move forward with this PR: I would at least like to see a proper unit test.

[kornicameister]

@hjacobs with all due respect and I am saying this with all best intentions in my heart. I'd see connexion progress and be maintained with the love it needs. It's just that it really requires some sort of revamp in quite a few places. I have noticed that a lot of protocols a core abstract classes make are not met and it is done by either aiohttp, flask or, perhaps, even both implementations.

[ddurham2]

@hjacobs: Unit test added. Works with new changes, fails without them.

[ddurham2]

buhbump?

[crertel-axios]

bomp

[hjacobs]

👍