Skip to content
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

fix(jersey3): Fix issue: server will not reregister after peer server restarts #1540

Open
wants to merge 1 commit into
base: 2.x
Choose a base branch
from

Conversation

ouaibsky
Copy link

@ouaibsky ouaibsky commented Apr 5, 2024

When we got a 404, the payload does not contains a json that can be map into an InstanceInfo

more detail with reprductibe example at: spring-cloud/spring-cloud-netflix#4220 (comment)

It's fixing issue:

@ouaibsky
Copy link
Author

Hello,
Can someone have a look please ?

@ouaibsky
Copy link
Author

ouaibsky commented Apr 24, 2024

Hi @howardyuan
Can you have a look please ?

@ouaibsky
Copy link
Author

Thx you @howardyuan
Could a maintainer take a look ? maybe @drobertduke ?

Thx in advance
Chris.

@RolandStaub
Copy link

@ouaibsky will this fix get merged soon, or are there other concerns ?

@ouaibsky
Copy link
Author

@ouaibsky will this fix get merged soon, or are there other concerns ?

I would like this to be merged, but looks like there is no maintainer to validate or ask question.
This fix works like a charm where I deploied it with a custom distribution.

@ouaibsky
Copy link
Author

ouaibsky commented Aug 7, 2024

Hi @paulbakker, I'm looking for a maintainer to review this PR.
Can you help me please ?
Thx
Ouaibsky

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants