-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
JerseyEurekaHttpClient issue with 404 responses in Spring Boot 3.x #1515
Comments
Hello I confirm this is a big issue when using spring-cloud-eureka-server, breaking the replication between 2 server instances (after one instance disappear and try to come back) It's a simple fix I can do but I'm not able to find the source code for eureka-client-jersey3 ? pom.xml from eureak-client-jersey3 is referencing this repository. Thx |
Hey |
PR is available here: #1540 Tested successfully with example provided at: spring-cloud/spring-cloud-netflix#4220 (comment) |
Spring Boot 3.x includes a payload to error responses, by default.
exmple:
On the other hand
JerseyEurekaHttpClient::sendHeartBeat
method tries to read and deserialise the response payload toInstanceInfo
, if it is not empty.Failure in deserialising the above error payload results in 400 (bad request) error and the client does not continue with the registration (which supposed to happened after receiving 404).
A similar issue in Spring Cloud Netflix has been fixed by updating RestTemplateTransportClientFactory and WebClientTransportClientFactory.
I believe a simple solution needs to be implemented for JerseyEurekaHttpClient.
Here is a sample stacktrace :
The text was updated successfully, but these errors were encountered: