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

Spring cloud version upgraded, loadBalancer replaced ribbon #4958

Merged
merged 44 commits into from
Mar 15, 2024

Conversation

ChengJie1053
Copy link
Member

@ChengJie1053 ChengJie1053 commented Nov 7, 2023

Spring cloud version upgraded and loadBalancer replaced ribbon

Compatible with nacos
image

image

@ChengJie1053 ChengJie1053 changed the title [WIP] Spring cloud version upgraded and loadBalancer replaced ribbon Spring cloud version upgraded and loadBalancer replaced ribbon Nov 28, 2023
<groupId>io.springfox</groupId>
<artifactId>springfox-spring-webmvc</artifactId>
<version>${springfox.version}</version>
</dependency>
Copy link
Contributor

Choose a reason for hiding this comment

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

is spring gateway use the springfox-spring-webmvc dependency?

<knife4j.version>2.0.9</knife4j.version>
<springfox.version>2.10.5</springfox.version>
<knife4j.version>3.0.3</knife4j.version>
<springfox.version>3.0.0</springfox.version>
Copy link
Contributor

Choose a reason for hiding this comment

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

can use knife4j 4.x, and springdoc replace?

@ChengJie1053 ChengJie1053 changed the title Spring cloud version upgraded and loadBalancer replaced ribbon Spring cloud version upgraded; loadBalancer replaced ribbon Jan 26, 2024
@ChengJie1053 ChengJie1053 changed the title Spring cloud version upgraded; loadBalancer replaced ribbon Spring cloud version upgraded, loadBalancer replaced ribbon Feb 26, 2024
pom.xml Outdated Show resolved Hide resolved
Copy link
Contributor

@jackxu2011 jackxu2011 left a comment

Choose a reason for hiding this comment

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

LGTM

@jackxu2011 jackxu2011 merged commit f876a73 into apache:master Mar 15, 2024
12 checks passed
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.

3 participants