`@inject` breaks `inspect.iscoroutinefunction`

[burritoatspoton]

After upgrade to 4.39.0, while asyncio.coroutines.iscoroutinefunction is preserved, inspect.iscoroutinefunction is not -- tested on Python 3.10.4:

from dependency_injector.wiring import inject
import inspect

async def foo(): pass

@inject
async def bar(): pass

print(inspect.iscoroutinefunction(foo))
print(inspect.iscoroutinefunction(bar))

This e.g. makes FastAPI incorrectly recognize bar as a "normal" function when used with dep = fastapi.Depends(bar) syntax in router dependencies.

[rmk135]

Hi @burritoatspoton ,

Thanks a lot for reporting the issue. I have Cythonized some parts of @inject decorator and it caused the problem. Seems like the problem is known and the roots go back to 2018: cython/cython#2273.

Working on the fix.

Best,
Roman

[rmk135]

Fixed in 4.39.1. The fix should be safe and most of the applied optimizations from 4.39.0 are still in place.

PS: here's the commit with the hotfix summary highlighted: cfed30c#diff-4af6e647307dbe71b243ef06bbbefaafa5c03b9a57b92bc8106cdd4a3e96a6e7R901-R913

[burritoatspoton]

Thank you for the quick fix!
Best,
Rafal