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

Routing by port #187

Merged
merged 1 commit into from
Dec 31, 2024
Merged

Conversation

hexiaofeng
Copy link
Collaborator

What is the purpose of the change?

Such a service call, “http://service-provider:8080/echo/”, requires routing according to the 8080 port.

Checklist

  • Make sure there is a GitHub_issue field for the change.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Write necessary unit-test to verify your logic correction. If the new feature or significant change is committed, please remember to add sample in joylive-demo project.
  • Make sure gitHub actions can pass. Why the workflow is failing and how to fix it?

@hexiaofeng hexiaofeng linked an issue Dec 31, 2024 that may be closed by this pull request
@hexiaofeng hexiaofeng self-assigned this Dec 31, 2024
@hexiaofeng hexiaofeng added the enhancement New feature or request label Dec 31, 2024
Copy link
Collaborator

@chenzhiguo chenzhiguo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chenzhiguo chenzhiguo merged commit 7a37415 into main Dec 31, 2024
1 check passed
@hexiaofeng hexiaofeng deleted the 186-support-routing-by-port-in-spring-cloud branch December 31, 2024 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support routing by port in spring cloud
2 participants