use sprintf un toString

Fixes elastic#4325
Eric013 · Dec 10, 2015 · aaaa6de · aaaa6de
1 parent aef6b5a
commit aaaa6de
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/logstash-core-event-java/src/main/java/com/logstash/Event.java b/logstash-core-event-java/src/main/java/com/logstash/Event.java
@@ -196,13 +196,13 @@ public Event clone()
     }
 
     public String toString() {
-        // TODO: until we have sprintf
-        String host = (String)this.data.getOrDefault("host", "%{host}");
-        String message = (String)this.data.getOrDefault("message", "%{message}");
+        // TODO: (colin) clean this IOException handling, not sure why we bubble IOException here
         try {
-            return getTimestamp().toIso8601() + " " + host + " " + message;
+            return (getTimestamp().toIso8601() + " " + this.sprintf("%{host} %{message}"));
         } catch (IOException e) {
-            return host + " " + message;
+            String host = (String)this.data.getOrDefault("host", "%{host}");
+            String message = (String)this.data.getOrDefault("message", "%{message}");
+            return (host + " " + message);
         }
     }