Skip to content

Escape asterisk character - #1722

Merged
ryanjbaxter merged 3 commits into
spring-cloud:mainfrom
ngocnhan-tran1996:escape-character
Aug 13, 2026
Merged

Escape asterisk character#1722
ryanjbaxter merged 3 commits into
spring-cloud:mainfrom
ngocnhan-tran1996:escape-character

Conversation

Signed-off-by: Tran Ngoc Nhan <ngocnhan.tran1996@gmail.com>
When a `@ConfigurationProperties` bean is re-bound, its properties are first reset to their class-level defaults so that values for properties that have been removed from the `Environment` do not linger.
This reset also recurses into nested `@ConfigurationProperties` objects (for example a nested object that has no setter).
To avoid descending into object graphs that are not configuration properties, the reset never recurses into JDK types or types in the standard `jakarta.*` and `javax.*` API namespaces.
To avoid descending into object graphs that are not configuration properties, the reset never recurses into JDK types or types in the standard `jakarta.\*` and `javax.*` API namespaces.

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.

The * in the java.* needs to be escaped too, right?

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.

I ran it on my local machine, and java.* doesn’t need to be escaped. But to be safe, I think we should escape it anyway.

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.

I was wrong, I fixed it and removed them, sorry about that!

== Configuring Individual LoadBalancerClients

Individual Loadbalancer clients may be configured individually with a different prefix `spring.cloud.loadbalancer.clients.<clientId>.*` where `clientId` is the name of the loadbalancer. Default configuration values may be set in the `spring.cloud.loadbalancer.*` namespace and will be merged with the client specific values taking precedence
Individual Loadbalancer clients may be configured individually with a different prefix `spring.cloud.loadbalancer.clients.<clientId>.\*` where `clientId` is the name of the loadbalancer. Default configuration values may be set in the `spring.cloud.loadbalancer.*` namespace and will be merged with the client specific values taking precedence

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.

Same here, I believe the * in spring.cloud.loadbalancer.* needs to be escaped

ngocnhan-tran1996 and others added 2 commits August 13, 2026 23:34
Signed-off-by: Tran Ngoc Nhan <ngocnhan.tran1996@gmail.com>
@github-project-automation github-project-automation Bot moved this to Todo in 2025.1.3 Aug 13, 2026
@ryanjbaxter ryanjbaxter added this to the 5.0.3 milestone Aug 13, 2026
@ryanjbaxter
ryanjbaxter merged commit 6d3a93b into spring-cloud:main Aug 13, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in 2025.1.3 Aug 13, 2026
@ngocnhan-tran1996
ngocnhan-tran1996 deleted the escape-character branch August 14, 2026 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants