-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
[ISSUE#12979] Develop a maintainer sdk. #13064
base: v3.0-develop
Are you sure you want to change the base?
[ISSUE#12979] Develop a maintainer sdk. #13064
Conversation
Thanks for your this PR. 🙏 感谢您提交的PR。 🙏 |
console/pom.xml
Outdated
@@ -77,6 +77,11 @@ | |||
<groupId>org.slf4j</groupId> | |||
<artifactId>jul-to-slf4j</artifactId> | |||
</dependency> | |||
<dependency> | |||
<groupId>com.alibaba.nacos</groupId> | |||
<artifactId>maintainer-client</artifactId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be nacos-maintainer-client?
console/pom.xml
Outdated
<dependency> | ||
<groupId>com.alibaba.nacos</groupId> | ||
<artifactId>maintainer-client</artifactId> | ||
<version>0.0.1-SNAPSHOT</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version follow parent
|
||
@Configuration | ||
@ConditionalOnProperty(value = Constants.NACOS_DEPLOYMENT_TYPE, havingValue = Constants.NACOS_DEPLOYMENT_TYPE_CONSOLE) | ||
public class NacosRemoteHandlerConfiguration { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Console相关的改动可以放在之后的PR里,先添加maintainer client,可以按照console 所需要的API先行添加。然后再补充一些基础的
* this is only use in nacos, Custom request type, essentially a GET request, Mainly used for GET request parameters | ||
* are relatively large,can not be placed on the URL, so it needs to be placed in the body. | ||
*/ | ||
public static final String GET_LARGE = "GET-LARGE"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个类型现在还在用吗?
* @author nkorange | ||
* @since 0.8.0 | ||
*/ | ||
public class HttpMethod { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个基础类是从nacos-client copy出来的吗?
Please do not create a Pull Request without creating an issue first.
What is the purpose of the change
it closes #12979
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily:
[ISSUE #123] Fix UnknownException when host config not exist
. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=true
to make sure basic checks pass. Runmvn clean install -DskipITs
to make sure unit-test pass. Runmvn clean test-compile failsafe:integration-test
to make sure integration-test pass.