Skip to content

Commit

Permalink
fix: rydder alle mdc nøkler før hver kjøring. kopierer til error task (
Browse files Browse the repository at this point in the history
  • Loading branch information
frode-carlsen authored May 31, 2020
1 parent bdbe497 commit a086263
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

import java.time.LocalDateTime;

import org.slf4j.MDC;

/** IdentRunnable som tar id og Runnable. */
class IdentRunnableTask implements IdentRunnable {
private final Long id;
Expand All @@ -16,6 +18,7 @@ class IdentRunnableTask implements IdentRunnable {

@Override
public void run() {
MDC.clear();
runnable.run();
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ class TaskManagerRunnableTask implements Runnable {

@Override
public void run() {
MDC.clear();

RunTask runSingleTask = newRunTaskInstance();
IdentRunnable errorCallback = null;
Expand All @@ -45,15 +46,16 @@ public void run() {
clearLogContext();
// dispose CDI etter bruk
TaskManagerGenerateRunnableTasks.CURRENT.destroy(runSingleTask);

// kjør etter at runTask er destroyed og logcontext renset
handleErrorCallback(errorCallback);
}
}

IdentRunnable lagErrorCallback(final RunTaskInfo taskInfo, final String callId, final Throwable fatal) {
Runnable errorCallback;
var mdcCopy = MDC.getCopyOfContextMap();
errorCallback = () -> {
MDC.setContextMap(mdcCopy); // later som vi fortsetter med samme MDC nøkler, men nå i ny tråd
final FatalErrorTask errorTask = TaskManagerGenerateRunnableTasks.CURRENT.select(FatalErrorTask.class).get();
try {
initLogContext(callId, taskInfo.getTaskType());
Expand All @@ -76,8 +78,7 @@ IdentRunnable lagErrorCallback(final RunTaskInfo taskInfo, final String callId,
}

void clearLogContext() {
TaskManagerGenerateRunnableTasks.LOG_CONTEXT.clear();
MDC.remove(CallId.CALL_ID);
MDC.clear();
}

void initLogContext(final String callId, String taskName) {
Expand Down

0 comments on commit a086263

Please sign in to comment.