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

[Feature] Support more configuration of nacos properties #326

Open
KomachiSion opened this issue Feb 20, 2024 · 2 comments
Open

[Feature] Support more configuration of nacos properties #326

KomachiSion opened this issue Feb 20, 2024 · 2 comments
Labels

Comments

@KomachiSion
Copy link
Collaborator

KomachiSion commented Feb 20, 2024

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 : alibaba/spring-cloud-alibaba#1124

@FeiXiangDouLi
Copy link

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

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

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

@KomachiSion
Copy link
Collaborator Author

KomachiSion commented Mar 25, 2024

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

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

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

实现类似 alibaba/spring-cloud-alibaba#1124 的效果, 但是参数的prefix应该和其他的参数保持一致,
应该是nacos.confignacos.discoveiry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants