Skip to content

Log4J2 configuration URLs should respect system settings #32735

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

Log4J2 configuration URLs should respect system settings #32735

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

It's currently possible to use URLs to load Log4j2 configuration. In Spring Boot 2.x we opened these URLs directly, but it would be better if we used Log4j2 mechanisms so that properties such as log4j2.trustStoreLocation are respected.
@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 philwebb self-assigned this Oct 14, 2022
philwebb pushed a commit that referenced this issue Oct 14, 2022
@rgoers @philwebb
Resolve URLs using Log4j2 mechanisms
5286785 
Update `Log4J2LoggingSystem` to that non file URLs are resolved using
Log4j2's `UrlConnectionFactory` mechanism rather than directly.

See gh-32735
philwebb added a commit that referenced this issue Oct 14, 2022
@philwebb
Polish 'Resolve URLs using Log4J2 mechanisms'
cec090c 
See gh-32735
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
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