spring-cloud / spring-cloud/spring-cloud-commons
Align ManagementServerPortUtils strategy from boot 2
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 751
- Forks
- 744
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 9
Description
ManagementServerPortUtils.java was ported from boot (as it was not public). It is likely old and not up to date.
It is used in two places:
https://github.com/spring-cloud/spring-cloud-commons/blob/master/spring-cloud-commons/src/main/java/org/springframework/cloud/client/serviceregistry/AbstractAutoServiceRegistration.java#L156
and
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with spring-cloud-commons/src/main/java/org/springframework/cloud/client/discovery/ManagementServerPortUtils.java and compare it with Spring Boot's ManagementPortType.java. Trace both usages in AbstractAutoServiceRegistration.java at the referenced lines, then determine the changes needed to align the utility's strategy with Boot's current behavior. Done means both usages follow the updated strategy.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java, spring-boot
- Domain
- backend, distributed-systems
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100