If a custom object responds to id method, show the id value, instea…

…d of showing "[OBJECT]"

lib/bugsnag/cleaner.rb

@@ -7,6 +7,7 @@ class Cleaner
     RECURSION = '[RECURSION]'.freeze
     OBJECT = '[OBJECT]'.freeze
     RAISED = '[RAISED]'.freeze
+    OBJECT_WITH_ID_AND_CLASS = '[OBJECT]: [Class]: %{class_name} [ID]: %{id}'.freeze
 
     def initialize(filters)
       @filters = Array(filters)
@@ -47,7 +48,12 @@ def traverse_object(obj, seen, scope)
         str = obj.to_s rescue RAISED
         # avoid leaking potentially sensitive data from objects' #inspect output
         if str =~ /#<.*>/
-          OBJECT
+          # Use id of the object if available
+          if obj.respond_to?(:id)
+            OBJECT_WITH_ID_AND_CLASS % { class_name: obj.class, id: obj.id }
+          else
+            OBJECT
+          end
         else
           clean_string(str)
         end

spec/cleaner_spec.rb

@@ -48,6 +48,16 @@ class Macaron; end
       expect(subject.clean_object(a)).to eq('[OBJECT]')
     end
 
+    it "cleans custom objects to show the id of the object if object responds to id method" do
+      class Macaron
+        def id
+          10
+        end
+      end
+      a = Macaron.new
+      expect(subject.clean_object(a)).to eq("[OBJECT]: [Class]: #{a.class.name} [ID]: #{a.id}")
+    end
+
     it "cleans up binary strings properly" do
       if RUBY_VERSION > "1.9"
         obj = "Andr\xc7\xff"