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

Add ScopedContext to version 2.25.0 #2834

Draft
wants to merge 2 commits into
base: 2.x
Choose a base branch
from
Draft

Add ScopedContext to version 2.25.0 #2834

wants to merge 2 commits into from

Conversation

ppkarwasz
Copy link
Contributor

This PR reintroduces ScopedContext into version 2.25.0.

We discussed about introducing a new log4j-context module, therefore it is not ready to be merged.

ppkarwasz and others added 2 commits August 12, 2024 16:51
This reverts the changes that introduces `ScopedContext`.
This PR restores the scoped context introduced in #2438
into the `2.25.x` branch.
@rgoers
Copy link
Member

rgoers commented Aug 13, 2024

FYI - I am fine with this. I have been working on log4j-context in the background but my day job activities have left me little time.

@ppkarwasz
Copy link
Contributor Author

FYI - I am fine with this. I have been working on log4j-context in the background but my day job activities have left me little time.

Sure, take your time. 😉

In this 2.25.x branch I just made a snapshot of the current status of the 2.x branch.

ppkarwasz added a commit to apache/logging-log4j-samples that referenced this pull request Oct 28, 2024
By using the new `log4j-convert-plugin-descriptor` tool introduced in
apache/logging-log4j-transform#139
this PR:

1. Adds a customized `use-log4j-core-jtl-minimal` GraalVM profile that
   only includes the plugins required by JSON Template Layout.
2. Generates the existent `use-log4j-core-minimal` GraalVM profile using
   a custom `Log4j2Plugins.json` file. The previous profile was edited
   manually.

Closes apache/logging-log4j2#2834.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants