-
Notifications
You must be signed in to change notification settings - Fork 1.9k
implementation of a SLF4J MDC context #403
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
Conversation
In case this is merged will add it to kotlin-logging docs |
I think it's a really useful addition must have! But I would question it's existence here rather than in kotlin-logging. |
I think that both here and in kotlin-logging this will be an integration module. @elizarov suggested: "We should really have it out-of-the box in a separate integration module." in #119 . |
We use the CoRoutines rxjava integration and stumbled across the necessity to make the SLF4J MDC Context within in the CoRoutine scope. We implemented a solution based on the the description here:
#119