Formatter added in Java Util Logging Handler

Update:
- Event message with string formatter issue is fixed by adding Formatter while sending events.

Author: bparmar-splunk

src/main/java/com/splunk/logging/HttpEventCollectorLoggingHandler.java

@@ -83,7 +83,10 @@
 import com.google.gson.Gson;
 import com.splunk.logging.hec.MetadataTags;
 
-import java.util.*;
+import java.util.HashMap;
+import java.util.LinkedHashMap;
+import java.util.Locale;
+import java.util.Map;
 import java.util.logging.*;
 
 /**
@@ -258,10 +261,19 @@ public void publish(LogRecord record) {
             isExceptionOccured = true;
         }
 
+        /*
+        Initializing a formatter for Java Util Logging.
+        This will be used when placeholders are used for event logging in log methods.
+         */
+        Formatter messageFormatter = getFormatter();
+        if (messageFormatter == null) {
+            messageFormatter = new SimpleFormatter();
+        }
+
         this.sender.send(
                 record.getMillis(),
                 record.getLevel().toString(),
-                record.getMessage(),
+                messageFormatter.formatMessage(record),
                 includeLoggerName ? record.getLoggerName() : null,
                 includeThreadName ? String.format(Locale.US, "%d", record.getThreadID()) : null,
                 null, // no property map available

src/test/java/HttpEventCollector_JavaLoggingTest.java

@@ -14,17 +14,18 @@
  * under the License.
  */
 
-import java.util.*;
-
 import com.google.gson.JsonObject;
 import com.google.gson.JsonPrimitive;
 import com.splunk.logging.HttpEventCollectorErrorHandler;
 import com.splunk.logging.HttpEventCollectorEventInfo;
-
 import org.apache.commons.lang3.StringUtils;
 import org.junit.Assert;
 import org.junit.Test;
 
+import java.util.ArrayList;
+import java.util.Date;
+import java.util.HashMap;
+import java.util.List;
 import java.util.logging.Logger;
 
 public final class HttpEventCollector_JavaLoggingTest {