-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support sentry-go? #85
Comments
The old raven-go was archived, so we'll move to new sentry-go. Should we create new sentry hook repository for the new sentry-go? |
That'd be cool. I've also posted in the sentry-go repo (getsentry/sentry-go#43) with some basic code I'm using right now, fwiw. |
For those who are looking for sentry-go based solution. My take on it: https://github.com/makasim/sentryhook |
@evalphobia is this WIP? |
Sorry, I've not created yet. If there are no similar alternatives and you and other developers want to use new sentry-go with logrus, I'll create a sentry library having minimum feature on a new github org (e.g. https://github.com/orgs/logrussentry/) and invite you guys. |
Is this project still active? We are currently using this hook and it's been incredibly useful so far but we would be keen that it takes advantage of latest Sentry features and development. |
I have been using this repo for years until I found it has some issues with flushing, makasim/sentryhook is good but I found sentry does have a logrus plugin built-in in the SDK repo: https://github.com/getsentry/sentry-go/tree/master/logrus and it is good enough for me, just need a simple converter, and it works perfect with flushing, just call native |
Looks like Sentry is migrating their go package from raven-go to sentry-go (https://github.com/getsentry/sentry-go). Is there any plan to support it? Thanks!
The text was updated successfully, but these errors were encountered: