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

Adds support for Log4j SpringLookup #30004

Closed
wants to merge 1 commit into from

Conversation

ppkarwasz
Copy link
Contributor

Since Log4j 2.14.0, configuration files can resolve properties from Spring's Environment.

While this is already possible with the custom Log4j2CloudConfigLoggingSystem, this PR enables this feature in the
standard Log4j2LoggingSystem.

Since Log4j 2.14.0, configuration files can resolve properties from
Spring's `Environment`.

While this is already possible with the custom
`Log4j2CloudConfigLoggingSystem`, this PR enables this feature in the
standard `Log4j2LoggingSystem`.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 27, 2022
@wilkinsona
Copy link
Member

Thanks for the proposal. Unfortunately, we’re not keen on the existing circular dependencies between Spring Boot and Log4j 2 and this will strengthen that cycle. We feel that is the wrong direction to take.

We’d prefer to break the cycle as proposed in #28909 by providing functionality that is equivalent to SpringLookup. This would also remove one of the differences between Spring Boot’s Logback and Log4j 2 integration.

Thanks again for the proposal.

@wilkinsona wilkinsona closed this Feb 27, 2022
@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 27, 2022
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.

3 participants