-
Notifications
You must be signed in to change notification settings - Fork 498
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
fix ReactiveWebServerFactoryAutoConfiguration not support sprintboot 1.x #927
Conversation
WalkthroughThe modification involves changes in the auto-configuration order for a Spring Boot application, specifically for reactive web server setups. The update removes a direct import and alters the Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #927 +/- ##
============================================
- Coverage 78.70% 78.49% -0.22%
- Complexity 836 849 +13
============================================
Files 164 167 +3
Lines 6796 6869 +73
Branches 1009 1016 +7
============================================
+ Hits 5349 5392 +43
- Misses 923 951 +28
- Partials 524 526 +2 ☔ View full report in Codecov by Sentry. |
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.
Actionable comments posted: 0
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- sofa-ark-parent/support/ark-springboot-integration/ark-springboot-starter/src/main/java/com/alipay/sofa/ark/springboot/ArkReactiveAutoConfiguration.java (2 hunks)
Additional comments not posted (2)
sofa-ark-parent/support/ark-springboot-integration/ark-springboot-starter/src/main/java/com/alipay/sofa/ark/springboot/ArkReactiveAutoConfiguration.java (2)
39-39
: Update to@AutoConfigureBefore
to use fully qualified class name.This change ensures compatibility with Spring Boot 1.x by avoiding direct import of
ReactiveWebServerFactoryAutoConfiguration
, which may not exist in earlier versions. Good use of the fully qualified name to maintain backward compatibility.
Line range hint
48-57
: Ensure proper resource management innettyReactiveWebServerFactory
.The method
nettyReactiveWebServerFactory
configures aNettyReactiveWebServerFactory
with custom routes and server customizers. It's crucial to verify that the resources (likeReactorResourceFactory
) are properly managed and released, especially whensetUseGlobalResources(false)
is used.
* 优化排包配置 (#918) * 优化显示 * fix as bizStateRecords * 更新显示 * 优化自动排包配置 * 更新排包描述 --------- Co-authored-by: leo james <leojames.googol@gmail.com> (cherry picked from commit 5a13e01) * runtime中静态合并部署时多模块并行执行createTempDir时异常 (#911) Co-authored-by: jiyunfei <jiyunfei@come-future.com> Co-authored-by: leo james <leojames.googol@gmail.com> (cherry picked from commit 67a0bdf) * add string contains (#926) (cherry picked from commit 07009d7) * fix ReactiveWebServerFactoryAutoConfiguration not support sprintboot 1.x (#927) (cherry picked from commit 41d3cda) * add biz jar url in biz module (#928) (cherry picked from commit e1f5872) * update netty version (#931) * update netty version * fix version netty --------- Co-authored-by: tangjiafu <tangjiafu@kuaishou.com> Co-authored-by: leojames <leojames.googol@gmail.com> (cherry picked from commit fb233cf) * update version to 3.0.6-SNAPSHOT --------- Co-authored-by: Lipeng <44571204+gaosaroma@users.noreply.github.com> Co-authored-by: FFF <31267018+1034323716@users.noreply.github.com> Co-authored-by: Laglangyue <35491928+laglangyue@users.noreply.github.com>
* 优化排包配置 (#918) * 优化显示 * fix as bizStateRecords * 更新显示 * 优化自动排包配置 * 更新排包描述 --------- Co-authored-by: leo james <leojames.googol@gmail.com> (cherry picked from commit 5a13e01) * runtime中静态合并部署时多模块并行执行createTempDir时异常 (#911) Co-authored-by: jiyunfei <jiyunfei@come-future.com> Co-authored-by: leo james <leojames.googol@gmail.com> (cherry picked from commit 67a0bdf) * add string contains (#926) (cherry picked from commit 07009d7) * fix ReactiveWebServerFactoryAutoConfiguration not support sprintboot 1.x (#927) (cherry picked from commit 41d3cda) * add biz jar url in biz module (#928) (cherry picked from commit e1f5872) * update netty version (#931) * update netty version * fix version netty --------- Co-authored-by: tangjiafu <tangjiafu@kuaishou.com> Co-authored-by: leojames <leojames.googol@gmail.com> (cherry picked from commit fb233cf) * update version to 3.1.4-SNAPSHOT * fix multi ark boot runner test --------- Co-authored-by: Lipeng <44571204+gaosaroma@users.noreply.github.com> Co-authored-by: FFF <31267018+1034323716@users.noreply.github.com> Co-authored-by: Laglangyue <35491928+laglangyue@users.noreply.github.com>
fix koupleless/koupleless#229
Summary by CodeRabbit