maintain non-String values in gsub when iterating through an Array

sbocahu · Jul 7, 2012 · 04ad0c3 · 04ad0c3
1 parent bde0aa3
commit 04ad0c3
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/lib/logstash/filters/mutate.rb b/lib/logstash/filters/mutate.rb
@@ -188,11 +188,12 @@ def gsub(event)
       if event[field].is_a?(Array)
         event[field] = event[field].map do |v|
           if not v.is_a?(String)
-            @logger.debug("gsub mutation is only applicable for Strings, " +
-                          "skipping", :field => field, :value => event[field])
-            next
+            @logger.warn("gsub mutation is only applicable for Strings, " +
+                          "skipping", :field => field, :value => v)
+            v
+          else
+            v.gsub(needle, replacement)
           end
-          v.gsub(needle, replacement)
         end
       else
         if not event[field].is_a?(String)