Skip to content

Fix missing environment prefix in child management context #46879

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

Fix missing environment prefix in child management context #46879

wants to merge 3 commits into from
+75 −1

Conversation

YongGoose
Copy link
Contributor

fixes #45858

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 18, 2025
@YongGoose YongGoose mentioned this pull request Aug 18, 2025
Open
@YongGoose
run format
2a74d15 
Signed-off-by: yongjunhong <[email protected]>
@YongGoose
run format
d787085 
Signed-off-by: yongjunhong <[email protected]>
@wilkinsona
Copy link
Member

Thanks for the PR, but this isn't the right fix. The bug is related to the system environment property source and the SpringApplication's environment prefix that's applied to that property source by SystemEnvironmentPropertySourceEnvironmentPostProcessor.

@wilkinsona wilkinsona closed this Aug 19, 2025
@wilkinsona wilkinsona added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 19, 2025
@YongGoose
Copy link
Contributor Author

Thanks for the PR, but this isn't the right fix. The bug is related to the system environment property source and the SpringApplication's environment prefix that's applied to that property source by SystemEnvironmentPropertySourceEnvironmentPostProcessor.

Thanks! I’ll look into the issue again and then open a PR.

@YongGoose YongGoose deleted the feature/45858 branch August 19, 2025 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
status: declined A suggestion or change that we don't feel we should currently apply
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Properties bound in the child management context ignore the parent's environment prefix
3 participants
@YongGoose @wilkinsona @spring-projects-issues