Merge pull request #6145 from microsoft/andxu-npe-uiloggerappender

andxu · web-flow · commit 5b98ab28d2de · 2021-11-25T21:13:06.000+08:00
Fix NPE issue in UILoggerAppender
diff --git a/Utils/hdinsight-node-common/src/com/microsoft/azure/hdinsight/common/logger/UILoggerAppender.java b/Utils/hdinsight-node-common/src/com/microsoft/azure/hdinsight/common/logger/UILoggerAppender.java
@@ -23,7 +23,7 @@ public void append(LoggingEvent event) {
             if (uiHelper != null) {
                 uiHelper.logError(
                         event.getRenderedMessage(),
-                        event.getThrowableInformation().getThrowable());
+                        event.getThrowableInformation() == null ? null : event.getThrowableInformation().getThrowable());
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ public void append(LoggingEvent event) {`
`23`	`23`	`if (uiHelper != null) {`
`24`	`24`	`uiHelper.logError(`
`25`	`25`	`event.getRenderedMessage(),`
`26`		`- event.getThrowableInformation().getThrowable());`
	`26`	`+ event.getThrowableInformation() == null ? null : event.getThrowableInformation().getThrowable());`
`27`	`27`	`}`
`28`	`28`	`}`
`29`	`29`