Merge pull request #891 from rollbar/wj-skip-non-json-keys

Expect to see and handle non-json objects

lib/rollbar/truncation/remove_any_key_strategy.rb

@@ -103,11 +103,21 @@ def extract_title_from_trace(trace)
       def data_keys
         @data_keys ||= {}.tap do |hash|
           data.keys.reject { |key| skip_keys.include?(key) }.each do |key|
-            size = dump(data[key]).bytesize
-            hash[key] = size
+            set_key_size(key, hash)
           end
         end
       end
+
+      def set_key_size(key, hash)
+        size = dump(data[key]).bytesize
+        hash[key] = size
+      rescue ::JSON::GeneratorError
+        hash[key] = 0 # don't try to truncate non JSON object
+
+        # Log it
+        truncation_key['non_json_keys'] ||= {}
+        truncation_key['non_json_keys'][key] = data[key].class
+      end
     end
   end
 end