failsafe-lib / failsafe

Fault tolerance and resilience patterns for the JVM
https://failsafe.dev
Apache License 2.0
4.17k stars 296 forks source link

Add feign module #329

Open jhalterman opened 2 years ago

jhalterman commented 2 years ago

Add a module supporting Feign. This is pretty straightforward to do for sync executions.

For async executions, it seems like Feign's AsyncFeign class is still experimental, and doesn't play nicely with an InvocationHandler since the async code uses a ThreadLocal and throws a NPE if called by an InvocationHandler from a different thread. It may not be worth adding async support until things mature a bit.

Tembrel commented 2 years ago

Clarifying: https://github.com/OpenFeign/feign