Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Introduce SPI Loading for Log4j ContextFieldSuppliers#199

Merged
KarstenSchnitter merged 4 commits into
SAP:release-4SAP/cf-java-logging-support:release-4from
KarstenSchnitter:release-4KarstenSchnitter/cf-java-logging-support:release-4Copy head branch name to clipboard
Feb 27, 2025
Merged

Introduce SPI Loading for Log4j ContextFieldSuppliers#199
KarstenSchnitter merged 4 commits into
SAP:release-4SAP/cf-java-logging-support:release-4from
KarstenSchnitter:release-4KarstenSchnitter/cf-java-logging-support:release-4Copy head branch name to clipboard

Conversation

@KarstenSchnitter

Copy link
Copy Markdown
Contributor

Adapt approach for Logback and introduce it for Log4j as well.
Provides SPI loading for ContextFieldSuppliers to generate
JSON format for log messages.

This allows declaration of ContextFieldSuppliers and
Log4jContextFieldSuppliers via SPI. The newly introduced
order is respected when the suppliers are executed. The old
mechanism with XML configuration is retained.

Signed-off-by: Karsten Schnitter <k.schnitter@sap.com>
Adapt the approach from Log4j making both implementations
uniform, where there is overlapping functionality.

Signed-off-by: Karsten Schnitter <k.schnitter@sap.com>
…cy-reduced-pom.xml

Signed-off-by: Karsten Schnitter <k.schnitter@sap.com>

@nicklas-dohrn nicklas-dohrn left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: Karsten Schnitter <k.schnitter@sap.com>
@KarstenSchnitter KarstenSchnitter merged commit b8f31c7 into SAP:release-4 Feb 27, 2025
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.