-
Notifications
You must be signed in to change notification settings - Fork 490
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 spring 3.2 #841
Merged
Merged
Merge spring 3.2 #841
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* suit sprignboot 3.2.0 * suit springboot 3.2.0 --------- Co-authored-by: 致节 <hzj266771@antgroup.com>
* Ark support Netty plugin * fix mvn issue * fix pom issue --------- Co-authored-by: Xuhao Wang <1023605039@qq.com> Co-authored-by: leo james <leojames.googol@gmail.com> Co-authored-by: yuanyuancin <yuancin@163.com> (cherry picked from commit 362f9a8)
…RELEASE to 2.5.12 in /sofa-ark-plugin/netty-ark-plugin (#785) * Bump org.springframework.boot:spring-boot-starter-webflux Bumps [org.springframework.boot:spring-boot-starter-webflux](https://github.com/spring-projects/spring-boot) from 2.1.0.RELEASE to 2.5.12. - [Release notes](https://github.com/spring-projects/spring-boot/releases) - [Commits](spring-projects/spring-boot@v2.1.0.RELEASE...v2.5.12) --- updated-dependencies: - dependency-name: org.springframework.boot:spring-boot-starter-webflux dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> * Update pom.xml --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: leo james <leojames.googol@gmail.com> (cherry picked from commit 6a9f471)
* support .* for exclude * support .* for exclude * support spring cloud * update to 2.2.5-SNAPSHOT * remove duplicate version properties setting * update to 2.2.5 (cherry picked from commit cefc679)
* support .* for exclude * support .* for exclude * update version (cherry picked from commit ef684f6)
* extend ArkNettyReactiveWebServerFactory * fix biz override master ReactorHttpHandlerAdapter * fix Port in use * fix base service faile when biz uninstalled * fix the biz service is still accessible after biz was uninstalled * change version to 2.2.5 * remove unuse code * disposableServer dispose when master stopping * fix ut,sofa-ark-springboot-starter dependent reactor-netty, change netty-handler version * fix ci check --------- Co-authored-by: yuanyuan <fengjun.zfj@antgroup.com> Co-authored-by: leo james <leojames.googol@gmail.com> (cherry picked from commit 361d445)
Co-authored-by: leo james <leojames.googol@gmail.com> (cherry picked from commit 584ac7c)
(cherry picked from commit 9c37315)
* support .* for exclude * support .* for exclude * enable override for plugin * optimize static import * enable classpath for plugin * add test for override * remove unused import * format * update 2.2.5-SNAPSHOT (cherry picked from commit 7de1384)
* feat: export plugin classes by default. * feat: export plugin classes by default. * feat: export plugin classes by default. * feat: export plugin package by default. * feat: export plugin package by default. * default add all packages for plugin * format --------- Co-authored-by: yanhuai.yh <yanhuai.yh@antgroup.com> Co-authored-by: leojames <leojames.googol@gmail.com> (cherry picked from commit 4ade973)
(cherry picked from commit a491814)
Co-authored-by: yuanyuan <fengjun.zfj@antgroup.com> (cherry picked from commit 52910e5)
* add ut * add tests * update test * update format * update test * update jarutils test * update tests * update tests * update web server test * update tests * update tests * update test * update test * update tests * update tests * update format * update tests * update test --------- Co-authored-by: LiuYu <ly109106@alibaba-inc.com> (cherry picked from commit bb17be2)
* Avoid biz logger context being attached early,support for logback * format * fix ut --------- Co-authored-by: yuanyuan <fengjun.zfj@antgroup.com> (cherry picked from commit 3fca4a6)
* default exclude pack rules * formar --------- Co-authored-by: yuanyuan <fengjun.zfj@antgroup.com> Co-authored-by: leo james <leojames.googol@gmail.com> (cherry picked from commit 7ec6184)
* add ut * add tests * update test * update format * update test * update jarutils test * update tests * update tests * update web server test * update tests * update tests * update test * update test * update tests * update tests * update format * update tests * update test * update tests * update test * Delete sofa-ark-parent/core/common/C:\temp dir\b\c directory * update test * update test --------- Co-authored-by: LiuYu <ly109106@alibaba-inc.com> (cherry picked from commit dfef361)
* support delegate from classLoaderHook for getResourceAsStream * rm guice-multibinding * remove guice multi binding lib * Revert "support delegate from classLoaderHook for getResourceAsStream" This reverts commit 5bd306b. * fix typo * remove guice-multibinding * fix format (cherry picked from commit 3bcf96f)
* support delegate from classLoaderHook for getResourceAsStream * Revert "support delegate from classLoaderHook for getResourceAsStream" This reverts commit 5bd306b. * fix typo * plugin default export package without activator, modify to set * fix format * export class for plugin as default (cherry picked from commit ef9584d)
* add ut * add tests * update test * update format * update test * update jarutils test * update tests * update tests * update web server test * update tests * update tests * update test * update test * update tests * update tests * update format * update tests * update test * update tests * update test * Delete sofa-ark-parent/core/common/C:\temp dir\b\c directory * update test * update test * update tesst * update test * update test * update test * update test * update test * update test * update test --------- Co-authored-by: LiuYu <ly109106@alibaba-inc.com> (cherry picked from commit c3192d0)
2. support SpringBoot 3.2.0 new LaunchedURLClassLoader class Co-authored-by: 致节 <hzj266771@antgroup.com> (cherry picked from commit 11f5507)
* add ut * add tests * update test * update format * update test * update jarutils test * update tests * update tests * update web server test * update tests * update tests * update test * update test * update tests * update tests * update format * update tests * update test * update tests * update test * Delete sofa-ark-parent/core/common/C:\temp dir\b\c directory * update test * update test * update tesst * update test * update test * update test * update test * update test * update test * update test * update test * update test * update test * update test --------- Co-authored-by: LiuYu <ly109106@alibaba-inc.com> (cherry picked from commit bc52a4c)
* support delegate from classLoaderHook for getResourceAsStream * Revert "support delegate from classLoaderHook for getResourceAsStream" This reverts commit 5bd306b. * fix typo * plugin default export package without activator, modify to set * fix format * export class for plugin as default * update to 2.2.6 * ark logger lazy init * format (cherry picked from commit 368a8be)
* update to 2.2.7 * fix windows mvn goal parse (cherry picked from commit 6a97493)
(cherry picked from commit 4f8c7d6)
lvjing2
reviewed
Feb 1, 2024
...-springboot-starter/src/test/java/com/alipay/sofa/ark/springboot/loader/ExplodedArchive.java
Outdated
Show resolved
Hide resolved
sofa-ark-parent/support/ark-maven-plugin/src/test/resources/application.properties
Outdated
Show resolved
Hide resolved
lvjing2
reviewed
Feb 1, 2024
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 3.1.x #841 +/- ##
=====================================
Coverage 0.00% 0.00%
=====================================
Files 153 153
Lines 6405 6405
Branches 961 961
=====================================
Misses 6405 6405 ☔ View full report in Codecov by Sentry. |
lvjing2
approved these changes
Feb 1, 2024
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.
LGTM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
support spring boot 3.2
Modification
Describe the idea and modifications you've done.
Result
Resolved or fixed #.
If there is no issue then describe the changes introduced by this PR.