topic Re: MDC Logging from methods called with Spring expression in Alfresco Archive

MDC Logging from methods called with Spring expression

jritter — Tue, 13 Sep 2016 16:47:04 GMT

Hi everybodyI am wondering whether it is possible to log the Activiti MDC Attributes in Log messages which are generated by our own code which is called by a Service Task. I've followed the instructions in the Activiti User Guide (http://www.activiti.org/userguide/#MDC), and the keys are showing up

Re: MDC Logging from methods called with Spring expression

warper — Wed, 14 Sep 2016 10:02:18 GMT

Hi Juerg!
Activiti engine sources (I scanned community edition version 5.20) contain only 1 file file directly working with MDC: org\activiti\engine\logging\LogMDC.java
It's used in few cases of exception handling. Something like this appears in catch blocks:
LogMDC.putMDCExecution(execution);
There are no calls of MDC, LogMDC in engine apart from exceptions…
There is also MDC context clearing function - LogMDC.clear(), which simply clears the whole MDC context (for current thread due to MDC context nature, but still the whole context instead of only activiti-related values).

You can set up MDC context directly calling LogMDC.putMDCExecution with current execution in your every delegate(listener).
As you probably know, you should also clean up MDC context, since it is thread-local variable and threads will be reused by executioners or web-service engine.

Re: MDC Logging from methods called with Spring expression

jritter — Wed, 14 Sep 2016 18:25:25 GMT

Hi Warper

Thanks a lot for your pointers, that really helped. I've set up a listener which invokes the puMDCExecution according to the event which has been triggered the listener. That seems to work, but I'm not sure yet whether this also holds when multiple workflows are running in parallel. I'll yet have to figure that out.

Thanks again and regards,
Juerg

Re: MDC Logging from methods called with Spring expression

mmalczewski — Wed, 21 Sep 2016 10:42:16 GMT

Putting a listener is one solution, but you can (and you will) forget about it. I'm experimenting with CommandContextFactory, which can be set in ProcessEngineConfigurationImpl:
<code>
class MDCCommandContextFactory extends CommandContextFactory {

@Override
public CommandContext createCommandContext(Command<?> cmd) {
return new MDCCommandContext(cmd, processEngineConfiguration);
}
}

class MDCCommandContext extends CommandContext {

public MDCCommandContext(Command<?> command, ProcessEngineConfigurationImpl processEngineConfiguration) {
super(command, processEngineConfiguration);
}

@Override
public void performOperation(AtomicOperation executionOperation, InterpretableExecution execution) {
LogMDC.putMDCExecution(execution);
try {
super.performOperation(executionOperation, execution);
}
finally {
LogMDC.clear();
}
}
}
</code>

This should set MDC information for each and every action, shouldn't it?

Re: MDC Logging from methods called with Spring expression

jritter — Wed, 21 Sep 2016 18:28:37 GMT

Thanks for your comment featur, that looks like a promising solution as well! I'll surely have a look at this too.

I'm curious, what kind of problems have you had with listeners?

Re: MDC Logging from methods called with Spring expression

mmalczewski — Wed, 19 Oct 2016 12:44:12 GMT

> I'm curious, what kind of problems have you had with listeners?
You must remember to use them. Somebody will forget about it at some point, that's more than sure in larger projects.