lib/raven/processor/sanitizedata.rb in sentry-raven-0.12.0 vs lib/raven/processor/sanitizedata.rb in sentry-raven-0.12.1

@@ -4,39 +4,33 @@
     INT_MASK = 0
     DEFAULT_FIELDS = %w(authorization password passwd secret ssn social(.*)?sec)
     VALUES_RE = /^\d{16}$/
 
     def process(value)
-      fields_re = /(#{(DEFAULT_FIELDS + @sanitize_fields).join("|")})/i
+      value.inject(value) { |memo,(k,v)|  memo[k] = sanitize(k,v); memo }
+    end
 
-      value.inject(value) do |value,(k,v)|
-        v = k if v.nil?
-        if v.is_a?(Hash) || v.is_a?(Array)
-          process(v)
-        elsif v.is_a?(String) && (json = parse_json_or_nil(v))
-          #if this string is actually a json obj, convert and sanitize
-          value = modify_in_place(value, [k,v], process(json).to_json)
-        elsif v.is_a?(Integer) && (VALUES_RE.match(v.to_s) || fields_re.match(k.to_s))
-          value = modify_in_place(value, [k,v], INT_MASK)
-        elsif VALUES_RE.match(v.to_s) || fields_re.match(k.to_s)
-          value = modify_in_place(value, [k,v], STRING_MASK)
-        else
-          value
-        end
+    def sanitize(k,v)
+      if v.is_a?(Hash)
+        process(v)
+      elsif v.is_a?(Array)
+        v.map{|a| sanitize(nil, a)}
+      elsif v.is_a?(String) && (json = parse_json_or_nil(v))
+        #if this string is actually a json obj, convert and sanitize
+        json.is_a?(Hash) ? process(json).to_json : v
+      elsif v.is_a?(Integer) && (VALUES_RE.match(v.to_s) || fields_re.match(k.to_s))
+        INT_MASK
+      elsif v.is_a?(String)  && (VALUES_RE.match(v.to_s) || fields_re.match(k.to_s))
+        STRING_MASK
+      else
+        v
       end
-      value
     end
 
     private
 
-    def modify_in_place(original_parent, original_child, new_child)
-      if original_parent.is_a?(Array)
-        index = original_parent.index(original_child[0])
-        original_parent[index] = new_child
-      elsif original_parent.is_a?(Hash)
-        original_parent[original_child[0]] = new_child
-      end
-      original_parent
+    def fields_re
+      @fields_re ||= /(#{(DEFAULT_FIELDS + @sanitize_fields).join("|")})/i
     end
   end
 end