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

log4j2: Use setContextData not setContextMap #156

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

paladox
Copy link
Contributor

@paladox paladox commented Apr 13, 2020

Also update the documentation for mdcProperties

Also update the documentation for mdcProperties
@hagbard
Copy link
Contributor

hagbard commented Apr 14, 2020

Thanks. I will keep this around since I want to rethink allowing the MDC stuff anyway (I accept I was wrong to be not allowing existing Log4J users to continue to use that mechanism - at least optionally).

Not sure I'd support Log4J MDC when using other backends though (but, maaaybe). I'll have a think about it.

@kevinb9n kevinb9n added the type=defect Bug, not working as expected label Apr 20, 2020
@cpovirk cpovirk added the P3 label Apr 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes P3 type=defect Bug, not working as expected
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants