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

fix: #326 修复 SpringCloud 下,远程配置文件修改(如Nacos),本地ConfigurationProperties 配置类没… #327

Merged
merged 2 commits into from
Dec 12, 2022

Conversation

noseew
Copy link

@noseew noseew commented Jun 23, 2022

‘Spring Cloud’ 中绝大多数对 ‘Spring Environment’ 操作的组件,应当比 ‘ConfigurationPropertiesRebinder’ 优先级要高,否则就会出现 ‘ConfigurationProperties’ 配置类更新不及时的问题。所以建议这里的 ‘RefreshScopeRefreshedEventListener’ 优先级不要设置成最低,只要比 ‘ConfigurationPropertiesRebinder’ 高即可;
-- from the technical team of YXT(yxt.com)

The vast majority of components in 'spring cloud' that operate on 'spring environment' should have higher priority than 'configurationpropertiesrebinder'. Otherwise, the 'configurationproperties' configuration class will not be updated in time. Therefore, it is recommended that the priority of 'refreshscoperefreshedeventlistener' should not be set to the lowest, as long as it is higher than 'configurationpropertiesrebinder';

…有解密的问题;同时保持 RefreshScopeRefreshedEventListener 优先级较低
@ulisesbocchio ulisesbocchio merged commit e7915dc into ulisesbocchio:master Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants