Allow non-string error message to be reported to sentry

the-spectator · the-spectator · commit 5617ca305a3e · 2023-11-24T11:18:37.000+05:30
diff --git a/sentry-ruby/lib/sentry/interfaces/single_exception.rb b/sentry-ruby/lib/sentry/interfaces/single_exception.rb
@@ -22,6 +22,7 @@ def initialize(exception:, stacktrace: nil)
         else
           exception.message || ""
         end
+      exception_message = exception_message.inspect unless exception_message.is_a?(String)
 
       @value = Utils::EncodingHelper.encode_to_utf_8(exception_message.byteslice(0..Event::MAX_MESSAGE_SIZE_IN_BYTES))
 
diff --git a/sentry-ruby/spec/sentry/client_spec.rb b/sentry-ruby/spec/sentry/client_spec.rb
@@ -209,6 +209,18 @@ def sentry_context
           expect(hash[:exception][:values][0][:value]).to eq("undefined method `[]' for nil:NilClass")
         end
       end
+
+      it "converts non-string error message" do
+        NonStringMessageError = Class.new(StandardError) do
+          def detailed_message(*)
+            { foo: "bar" }
+          end
+        end
+
+        event = subject.event_from_exception(NonStringMessageError.new)
+        expect(event).to be_a(Sentry::ErrorEvent)
+        expect(Sentry::Event.get_message_from_exception(event.to_hash)).to match("NonStringMessageError: {:foo=>\"bar\"}")
+      end
     end
 
     it "sets threads interface without stacktrace" do
