Fixes #475. Set context before setting JsonFactory in CompositeJsonFormatter

philsttr · philsttr · commit 8b62c210b7de · 2021-01-09T09:24:18.000-08:00
diff --git a/src/main/java/net/logstash/logback/composite/CompositeJsonFormatter.java b/src/main/java/net/logstash/logback/composite/CompositeJsonFormatter.java
@@ -102,8 +102,8 @@ public void start() {
             addError("No providers configured");
         }
         jsonFactory = createJsonFactory();
-        jsonProviders.setJsonFactory(jsonFactory);
         jsonProviders.setContext(context);
+        jsonProviders.setJsonFactory(jsonFactory);
         jsonProviders.start();
         started = true;
     }

Original file line number	Diff line number	Diff line change
`@@ -102,8 +102,8 @@ public void start() {`
`102`	`102`	`addError("No providers configured");`
`103`	`103`	`}`
`104`	`104`	`jsonFactory = createJsonFactory();`
`105`		`- jsonProviders.setJsonFactory(jsonFactory);`
`106`	`105`	`jsonProviders.setContext(context);`
	`106`	`+ jsonProviders.setJsonFactory(jsonFactory);`
`107`	`107`	`jsonProviders.start();`
`108`	`108`	`started = true;`
`109`	`109`	`}`