Skip to content

Add support 'log4j.configurationFile' system property #32730

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
Tracked by #32578
philwebb opened this issue Oct 14, 2022 · 0 comments
Closed
Tracked by #32578

Add support 'log4j.configurationFile' system property #32730

philwebb opened this issue Oct 14, 2022 · 0 comments
Assignees
@philwebb
Labels
type: enhancement A general enhancement
Milestone
3.0.0-RC1

Comments

@philwebb
Copy link
Member

Currently Log4J2LoggingSystem contains a hard-coded set of supported files but doesn't detect if the log4j.configurationFile system property is being used. For completeness we should include any file specified by this property.
@philwebb philwebb self-assigned this Oct 14, 2022
@philwebb philwebb added the type: enhancement A general enhancement label Oct 14, 2022
@philwebb philwebb added this to the 3.0.0-RC1 milestone Oct 14, 2022
philwebb pushed a commit that referenced this issue Oct 14, 2022
@rgoers @philwebb
Support 'log4j.configurationFile' system property
64eb36b 
Update `Log4J2LoggingSystem.getStandardConfigLocations()` so that
any configured 'log4j.configurationFile' system property is also
included as a location.

See gh-32730
philwebb added a commit that referenced this issue Oct 14, 2022
@philwebb
Polish 'Support 'log4j.configurationFile' system property'
a08a637 
See gh-32730
philwebb added a commit that referenced this issue Oct 14, 2022
@philwebb
Merge pull request for #32730 from rgoers
66e097e 
Closes gh-32731
philwebb added a commit that referenced this issue Oct 14, 2022
@philwebb
Merge pull request for #32578 from rgoers
b12fda2 
* pr/32578:
  Document Log4j2 extensions
  * Merge pull request for #32735:
    Polish 'Resolve URLs using Log4J2 mechanisms'
    Resolve URLs using Log4j2 mechanisms
  * Merge pull request for #32734:
    Polish 'Support profile specific Log4j2 configuration'
    Support profile specific Log4j2 configuration
  * Merge pull request for #32733:
    Polish 'Add Log4J2 PropertySource backed by the Spring Environment'
    Add Log4J2 PropertySource backed by the Spring Environment
  * Merge pull request for #32732:
    Polish 'Support Log4J2 string lookups from the Spring Environment'
    Support Log4J2 string lookups from the Spring Environment
  * Merge pull request for #32730:
    Polish 'Add Spring Environment to LoggerContext'
    Add Spring Environment to LoggerContext
  * Merge pull request for #32730:
    Polish 'Support 'log4j.configurationFile' system property'
    Support 'log4j.configurationFile' system property

Closes gh-32578
@philwebb philwebb mentioned this issue Oct 14, 2022
Closed
6 tasks
izeye added a commit to izeye/spring-boot that referenced this issue Oct 15, 2022
@izeye
Fix Log4J2LoggingSystemTests.configLocationsWithConfigurationFileSyst…
a000d74 
…emProperty()

See spring-projectsgh-32730
@izeye izeye mentioned this issue Oct 15, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philwebb philwebb

Labels
type: enhancement A general enhancement
Projects
None yet
Milestone
3.0.0-RC1
Development

No branches or pull requests
1 participant
@philwebb