getsentry / raven-go

Sentry client in Go

https://sentry.io

BSD 3-Clause "New" or "Revised" License

561 stars 148 forks source link

Pass full error context to sentry #185

Closed bfosberry closed 6 years ago

bfosberry commented 6 years ago

Currently the inner cause error is extracted to provide sentry with a stacktrace. This potentially wrapped error does not have useful wrapped error messaging giving context to the error and the code using it.

This change passes the error text from the original error rather than the wrapped cause.

Resolves #176

mattrobenolt commented 6 years ago

Sounds ok to me. I don't know much around this stuff, so blindly trusting you. :)

bfosberry commented 6 years ago

yolo, thanks!

On Tue, Apr 24, 2018 at 9:35 AM Matt Robenolt notifications@github.com wrote:

Sounds ok to me. I don't know much around this stuff, so blindly trusting you. :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/getsentry/raven-go/pull/185#issuecomment-383998135, or mute the thread https://github.com/notifications/unsubscribe-auth/ABeyj7kd_CrOONVGfReVuJFaf-Op9jnAks5tr1QzgaJpZM4TiAac .