-
Notifications
You must be signed in to change notification settings - Fork 784
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
Issue with registering method parameter hints #1021
Comments
@CC007 You need to add This is a simple and fast workaround for the issue, however, I will also take a look at registering reflection hints for classes used in OF method signatures automatically. |
This should actually work already as per 06192a5. Will need to verify why it doesn't work in the sample. |
Given that we can reasonably assume that any method defined in either a @FeignClient interface or a supertype interface is supposed to be part of the API, we should register all such methods for reflection and not only the ones declared for the annotated interface. That will fix the issue. |
Initially reported in #860 (comment).
Calling client generated with the sample: https://github.com/CodeQualIT/CQITFunctions, causes
feign.codec.EncodeException: Could not write request: no suitable HttpMessageConverter found for request type [nl.cqit.function.poc.java.helloworld.api.model.Person] and content type [application/json]
.Possibly related to class generation.
CC @CC007
The text was updated successfully, but these errors were encountered: