Fix breadcrumb serialization error to be an object (#2584)

Author: sl0thentr0py

CHANGELOG.md

@@ -9,6 +9,8 @@
 
 - Gracefully fail on malformed utf-8 breadcrumb message ([#2582](https://github.com/getsentry/sentry-ruby/pull/2582))
   - Fixes [#2376](https://github.com/getsentry/sentry-ruby/issues/2376)
+- Fix breadcrumb serialization error message to be an object ([#2584](https://github.com/getsentry/sentry-ruby/pull/2584))
+  - Fixes [#2478](https://github.com/getsentry/sentry-ruby/issues/2478)
 
 ## 5.23.0
 

sentry-ruby/lib/sentry/breadcrumb.rb

@@ -70,7 +70,7 @@ def serialized_data
           MSG
         end
 
-        DATA_SERIALIZATION_ERROR_MESSAGE
+        { error: DATA_SERIALIZATION_ERROR_MESSAGE }
       end
     end
   end

sentry-ruby/spec/sentry/breadcrumb_buffer_spec.rb

@@ -69,7 +69,7 @@
       expect(result[0][:data]).to eq({ "name" => "John", "age" => 25 })
       expect(result[1][:category]).to eq("bar")
       expect(result[2][:category]).to eq("baz")
-      expect(result[2][:data]).to eq("[data were removed due to serialization issues]")
+      expect(result[2][:data][:error]).to eq("[data were removed due to serialization issues]")
     end
   end
 end

sentry-ruby/spec/sentry/breadcrumb_spec.rb

@@ -104,7 +104,7 @@
 
       expect(result[:category]).to eq("baz")
       expect(result[:message]).to eq("I cause issues")
-      expect(result[:data]).to eq("[data were removed due to serialization issues]")
+      expect(result[:data][:error]).to eq("[data were removed due to serialization issues]")
       expect(stringio.string).to match(/can't serialize breadcrumb data because of error: nesting of 10 is too deep/)
     end
 
@@ -113,7 +113,7 @@
 
       expect(result[:category]).to eq("cow")
       expect(result[:message]).to eq("I cause too much recursion")
-      expect(result[:data]).to eq("[data were removed due to serialization issues]")
+      expect(result[:data][:error]).to eq("[data were removed due to serialization issues]")
       expect(stringio.string).to match(/can't serialize breadcrumb data because of error: nesting of 10 is too deep/)
     end
   end