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

Merge Develop #7312

Conversation

li-xiao-shuang
Copy link
Collaborator

Please do not create a Pull Request without creating an issue first.

What is the purpose of the change

XXXXX

Brief changelog

XX

Verifying this change

XXXX

Follow this checklist to help us incorporate your contribution quickly and easily:

  • Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
  • Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
  • 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, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
  • Run mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=true to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test to make sure integration-test pass.

brotherlu-xcq and others added 30 commits October 8, 2021 09:27
…st class (alibaba#7042)

* delete unit test case that already exist in HttpUtilsTest

* move unit test case to QueryTest

* move unit test case to ResponseHandlerTest

* delete empty test class
…baba#7133)

* [ISSUE alibaba#5092] add unit tests for constant in nacos-config

* [ISSUE alibaba#5092] add license for CounterModeTest.java
* can be replaced with lambda

* can be replaced with <>
* Move ability classes.

* Add ServerAbilityInitializer and RemoteAbilityInitializer

* Add NamingAbilityInitializer.

* Use ServerAbilityInitializer replace old init ability.

* For pmd.
* fix PushAckRequest mismatch problem.

* add doc.

* add doc.
* Add rest api v2 in core module

* merge cluster listNodes methods

* ut

* add beta annotation
* Add service rest api for v2.

* For PMD.
* replace with lambda

* replace with lambda
* can be replaced with lambda

* can be replaced with <>

* replace with lambda

* delete unused import
- Add catch block, return getBytes result if base64 decode failed
- Add JwtTokenManager unit test

Closes alibaba#7203
* Fix alibaba#6973 problem. customer istio sync hostname

* Fix alibaba#6973 problem. add domain suffix

* Fix alibaba#6973 problem. add domain suffix config

* Fix alibaba#6973 problem. run clean install and code format
onewe and others added 18 commits November 16, 2021 11:13
…nfig (alibaba#7206)

* [ISSUE alibaba#5092] add unit test for ConfigServletInner in nacos-config

* [ISSUE alibaba#5092] update ConfigServletInnerTest in nacos-config
… persistent). (alibaba#7239)

* Make one service keep one kind of instance (ephemeral or persistent).

* Fix IT error
…acos-co… (alibaba#7271)

* [ISSUE alibaba#5092] add unit test for utils about group\log\MD5 in nacos-config

* [ISSUE alibaba#5092] format unit test for utils about group\log\MD5 in nacos-config
…r the client module. Part 2 (alibaba#7256)

* add Constants.java

* Optimize the constants to the logging package for the client module
…the client module. Part 5 (alibaba#7259)

* add Constants.java

* Optimize the constants to the utils package for the client module
@li-xiao-shuang li-xiao-shuang merged commit 28caf87 into alibaba:summer2021_issue#5695 Nov 29, 2021
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.