apolloconfig / apollo-java

Apollo Java Clients
Apache License 2.0
40 stars 72 forks source link

fix: After the attribute is changed, increase the priority of spring … #48

Closed xingzhuipingye closed 9 months ago

xingzhuipingye commented 11 months ago

…attribute setting.

issue: https://github.com/apolloconfig/apollo/issues/5045

What's the purpose of this PR

springboot项目中,在apollo配置中心修改配置后,可以使用两种方式获得更改后的key 1、使用https://github.com/value("${configKey}"),每次使用方法获取属性就可以获得新的值 2、@ApolloConfigChangeListener,在回调方法ConfigChangeEvent中可以获取新的值

基于代码的简洁性,我在代码中监听了某个key,在回调中使用了https://github.com/value修饰的属性值,而不是用ConfigChange.getNewValue,在apollo中修改值后获取到了老的值

Which issue(s) this PR fixes:

Fixes # 提高spring修改properties的回调赋值优先级,在用户属性变更回调前,spring中的属性值已为最新

github-actions[bot] commented 11 months ago

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


wangfei seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request

stale[bot] commented 10 months ago

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 14 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

stale[bot] commented 9 months ago

This pull request has been automatically closed because it has not had activity in the last 14 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!