👋 Hi from the 
team.
I'm attempting to migrate from sentry-raven to sentry-ruby:
With Raven we performed some basic client-side sanitization of the exception message:
https://github.com/dependabot/dependabot-core/blob/65f1c2d24e88ac57ed8df980871446deb5ea2d94/updater/lib/dependabot/sentry.rb#L5-L21
The unit tests illustrate how it works:
https://github.com/dependabot/dependabot-core/blob/65f1c2d24e88ac57ed8df980871446deb5ea2d94/updater/spec/dependabot/sentry_spec.rb#L24-L91
Migrating this from Raven to sentry-ruby's before_send is straightforward. However, it looks like sentry-ruby doesn't expose :value on SingleExceptionInterface:
|
attr_reader :type, :value, :module, :thread_id, :stacktrace |
I see a recent change was made to expose :values on ExceptionInterface:
So I'm guessing there's no technical reason this can't be made publicly writable? Just no one has raised a feature request for it yet?
👋 Hi from the
team.
I'm attempting to migrate from
sentry-raventosentry-ruby:sentry-raventosentry-rubydependabot/dependabot-core#7553With Raven we performed some basic client-side sanitization of the exception message:
https://github.com/dependabot/dependabot-core/blob/65f1c2d24e88ac57ed8df980871446deb5ea2d94/updater/lib/dependabot/sentry.rb#L5-L21
The unit tests illustrate how it works:
https://github.com/dependabot/dependabot-core/blob/65f1c2d24e88ac57ed8df980871446deb5ea2d94/updater/spec/dependabot/sentry_spec.rb#L24-L91
Migrating this from Raven to
sentry-ruby'sbefore_sendis straightforward. However, it looks likesentry-rubydoesn't expose:valueonSingleExceptionInterface:sentry-ruby/sentry-ruby/lib/sentry/interfaces/single_exception.rb
Line 14 in 7764384
I see a recent change was made to expose
:valuesonExceptionInterface::valuesinExceptionInterface#1843So I'm guessing there's no technical reason this can't be made publicly writable? Just no one has raised a feature request for it yet?