hfaran / Tornado-JSON

A simple JSON API framework based on Tornado
http://tornado-json.readthedocs.org/
MIT License
273 stars 60 forks source link

Add support for async def #112

Closed cdsboy closed 5 years ago

cdsboy commented 5 years ago

This fixes the TypeError: Object of type 'coroutine' is not JSON serializable found when running Tornado 6.0 and using python 3 async def methods.

hfaran commented 5 years ago

Thanks for the PR. There are three failing tests - could you take a look at those?

Also, if this is the only issue to be fixed for Tornado 6.0 then I'll follow up and "officially" mark Tornado-JSON as supporting 6.0 (since as per the current requirements and testbed, only 5 and below is supported).

hfaran commented 5 years ago

Superseded by #113. Closing this.

coveralls commented 2 years ago

Coverage Status

Coverage decreased (-62.02%) to 35.223% when pulling 16e63f3e1513cb4ffde7f38ccdf5fd5c54a9a1e7 on cdsboy:coroutines into 8d8b35ff77f13cb3ab1a606bd2083b26cc69c54f on hfaran:master.