-
Notifications
You must be signed in to change notification settings - Fork 789
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
Multiple implementations of ResponseInterceptor fail to trigger #1006
Comments
Hello @marcdv, thanks for creating an issue. Please provide a minimal, complete, verifiable example that reproduces the issue, as a link to a GitHub repo with an executable app or tests. Please learn how to properly format code and logs. |
If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed. |
Please review https://github.com/marcdv/testFeign for an example Steps to reproduce
|
Thanks, @marcdv. That's actually never been supported and also the docs only show a way to provide one interceptor, so this would be an enhancement. Since this project is now in maintenance mode (suggested replacement: Spring Interface Clients), we are not planning on adding new features ourselves, but we still review small community PR. I see @galaxy-sea has submitted a PR for this, but it has breaking changes, so can only be considered for a major release and I will mark it as such, however at this point it has not been decided if there will be another major of this project. |
Hey @OlgaMaciaszek 👋🏻 , just asking, as I've seen you mention that Could you maybe point me to Docs where this is explained? Thanks in advance :) P.S.: |
Here a fixed linked: Spring Interface Clients. |
Hi @askingcat . That's strange. The link works for me. Can you please try it again (maybe the site was temporarily down)? Here it is, in the warning section: https://docs.spring.io/spring-cloud-openfeign/reference/ |
Thank you for sharing that link! :) Still off-topic, but regarding the link to the Spring Interface Client Docs: |
Thanks @LouisXhaferi, right, possibly something cached on my end. This should work: https://docs.spring.io/spring-framework/reference/integration/rest-clients.html#rest-http-interface |
Describe the bug
I have 2 implementations of
ResponseInterceptor
, both fail to trigger in4.1.0
. If I remove one of them, then it works.The text was updated successfully, but these errors were encountered: