Skip to content

Move field ContextRefresher.REFRESH_ARGS_PROPERTY_SOURCE down to LegacyContextRefresher#1450

Merged
ryanjbaxter merged 1 commit intospring-cloud:mainfrom
quaff:patch-11
Jan 23, 2025
Merged

Move field ContextRefresher.REFRESH_ARGS_PROPERTY_SOURCE down to LegacyContextRefresher#1450
ryanjbaxter merged 1 commit intospring-cloud:mainfrom
quaff:patch-11

Conversation

@quaff
Copy link
Copy Markdown
Contributor

@quaff quaff commented Jan 21, 2025

It's specific to LegacyContextRefresher.

…cyContextRefresher

It's specific to LegacyContextRefresher.

Signed-off-by: Yanming Zhou <[email protected]>
postProcessor.postProcessEnvironment(environment, application);
}

if (environment.getPropertySources().contains(REFRESH_ARGS_PROPERTY_SOURCE)) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this removed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because this PropertySource is added by LegacyContextRefresher, It's not involved if ConfigDataContextRefresher is using.

@ryanjbaxter ryanjbaxter merged commit 6c352b8 into spring-cloud:main Jan 23, 2025
@ryanjbaxter ryanjbaxter added this to the 4.1.6 milestone Jan 23, 2025
ryanjbaxter pushed a commit that referenced this pull request Jan 23, 2025
…cyContextRefresher (#1450)

It's specific to LegacyContextRefresher.

Signed-off-by: Yanming Zhou <[email protected]>
@spencergibb spencergibb moved this to Done in 2024.0.1 Mar 17, 2025
@spencergibb spencergibb moved this to Done in 2023.0.6 Mar 17, 2025
@spencergibb spencergibb moved this to Done in 2025.1.0-M1 Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

No open projects
Status: Done
Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants