-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Broken with Feign 12.5+ #2126
Comments
I see someone suggested this in one of the comments of another issue with same cause: However, important it gets back ported to 17.x as we are a few who are stuck in older java versions =/ |
I guess another option could be to extend internalBuild() instead of build() (as they made build final) |
I made a pull request addressing the issue here #2127 |
Unfortunately it's difficult for me to still support 1.7.x: I'm so busy at work at the moment, that it's difficult for me to find time to merge the changes from 2.x back to 1.7.x. :( |
I understand =) Not sure I can be of much aid there, I have created 3 PRs, one for each branch but was very long time ago I tried to get something published in a public repo. Well in the meantime maybe you can merge the change based on the master branch PR and get the fix for all current version users of resilience4j to get it to work. Its quite nasty as it does not produce any errors or warnings or anything as of now, it just dont work, so if you dont have good tests verifying the fallbacks etc, you wouldn't notice. Regarding 1.7.x in my personal case, we have a central middleware library that controls it all and I could live with a custom hack there. |
Resilience4j version: 1.7.x (cant run java17) (but very likely same issue in latest as same builder pattern is applied)
Java version: java11
Thanks for raising a Resilience4j issue.
I opened up this issue with Feign today:
OpenFeign/feign#2341
They have in Feign 12.5 onwards made the builder return a new builder at some point and the Resilience4jFeign.Builder is effectively not used when hitting build..
To workaround in my project I have injected the following class in your package:
Where I call PatchedResilience4jFeign.setInvocationHandler() just before I build the builder =)
Cant be done from my packages as DecoratorInvocationHandler is not public
The text was updated successfully, but these errors were encountered: