search

nacos-group / nacos-spring-boot-project

Nacos ECO Project for Spring Boot
Apache License 2.0
783 stars 423 forks source link

[Feature] Support more configuration of nacos properties #326

Open KomachiSion opened 6 months ago

KomachiSion commented 6 months ago

Nacos spring boot starter only support some of nacos client properties. But Nacos client support more properties in 2.X version such as namingPushEmptyProtection . And spring boot starter can't support them.

Prefer Nacos spring boot starter supports them like spring cloud alibaba.

refer implementation PR in SCA : https://github.com/alibaba/spring-cloud-alibaba/pull/1124

FeiXiangDouLi commented 6 months ago

我想尝试解决，请问目前有对应的方案或者具体要达到什么效果的目标文档吗？

是否只是按 alibaba/spring-cloud-alibaba#1124 一样，只是需要把环境变量中 spring.cloud.nacos.discovery 开头NacosDiscoveryProperties中没有的参数，放到properties中即可。

还是显式定义到NacosDiscoveryProperties中，进行加载配置。

KomachiSion commented 5 months ago

我想尝试解决，请问目前有对应的方案或者具体要达到什么效果的目标文档吗？

是否只是按 alibaba/spring-cloud-alibaba#1124 一样，只是需要把环境变量中 spring.cloud.nacos.discovery 开头NacosDiscoveryProperties中没有的参数，放到properties中即可。

还是显式定义到NacosDiscoveryProperties中，进行加载配置。

实现类似 https://github.com/alibaba/spring-cloud-alibaba/pull/1124 的效果， 但是参数的prefix应该和其他的参数保持一致， 应该是nacos.config 和 nacos.discoveiry.