koupleless / koupleless

modular dev framework and serving platform to enable app evolve from monolithic to microservices and also serverless smoothly. 模块化研发框架与运维调度系统,帮助应用解决研发运维系列痛点问题,省资源、秒级启动、灵活部署、快速需求交付等
https://koupleless.io
Apache License 2.0
164 stars 35 forks source link

部分获取基座 application.properties 配置的读取方式误用了 sofaArk 里的 EnvironmentUtils 工具类(主要从 jvm 参数读取) #276

Open lvjing2 opened 2 months ago

lvjing2 commented 2 months ago

问题描述

部分获取基座 application.properties 配置的读取方式误用了 sofaArk 里的 EnvironmentUtils 工具类(主要从 jvm 参数读取)

image

问题解决

通过获取基座 spring applicationContext 的 environment,然后获取 application.properties

shichaoyuan commented 2 months ago

EnvironmentUtils.getProperty(WITH_ALL_BIZ_READINESS, "false") 这一处看起来比较简单,直接将 appCtx autowired 进来就可以,默认就是基座的

String httpPort = EnvironmentUtils.getProperty(HTTP_PORT_ATTRIBUTE); 这里是 ArkletComponentRegistry 静态初始化时触发的,appCtx不太方便传进来,有啥建议?

lvjing2 commented 1 month ago

暂时还没有特别的想法。

lvjing2 commented 1 month ago

目前在考虑 arklet-core 是否需要接触 springboot 的依赖,也许 arklet-core 与 arklet-springboot-starter 可以考虑合并在一起

shichaoyuan commented 1 month ago

目前在考虑 arklet-core 是否需要接触 springboot 的依赖,也许 arklet-core 与 arklet-springboot-starter 可以考虑合并在一起

感觉是可以合在一起。

  1. 共享强绑定springboot
  2. arklet-core中的Guice与spring依赖注入是重叠的
lvjing2 commented 1 month ago

是的,有兴趣可以 self-assign 下哈。

shichaoyuan commented 1 month ago

是的,有兴趣可以 self-assign 下哈。

assign me

我尝试合并一下