Skip to content

add 'spring.config.intergration-location' with 'classpath*:' support #25082

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

Closed
wants to merge 2 commits into from

Conversation

stategen
Copy link

@stategen stategen commented Feb 2, 2021

The feature is adding a lowest argrement '--spring.config.intergration-location' which value support 'classpath*:' .
In some cases, the project integrates jars with an uncertain number, such as message subscribers. This can make a truly dynamic spring-stream and zero configuration for providers

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 2, 2021
@wilkinsona
Copy link
Member

wilkinsona commented Feb 2, 2021

Thanks for the proposal, @stategen. We don't want to add support for loading configuration from dependencies' jars but this isn't quite how we'd like to go about it. Please see #24688 for details. If you'd like to use what you have proposed here rather than waiting for #24688 to be implemented, you should be able to plug in something similar using an EnvironmentPostProcessor. Thanks again for the suggestion.

@wilkinsona wilkinsona closed this Feb 2, 2021
@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 Feb 2, 2021
@wilkinsona
Copy link
Member

I see this was also proposed as #25080. @stategen sorry that we couldn't merge your changes. Please don't spam the repository with duplicate pull requests.

@stategen
Copy link
Author

stategen commented Feb 2, 2021

@wilkinsona ,thks and sorry, I'm not familiar with the operation of GitHub. My first submission failed because of the formatting problem. I thought it was not a submission, so I submitted it again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: declined A suggestion or change that we don't feel we should currently apply
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants