Skip to content
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

Open
danqing opened this issue Jun 14, 2019 · 7 comments
Open

Support sentry-go? #85

danqing opened this issue Jun 14, 2019 · 7 comments
Assignees

Comments

@danqing
Copy link

danqing commented Jun 14, 2019

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!

@evalphobia evalphobia self-assigned this Aug 22, 2019
@evalphobia
Copy link
Owner

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?

@danqing
Copy link
Author

danqing commented Aug 23, 2019

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.

@makasim
Copy link

makasim commented Feb 12, 2020

For those who are looking for sentry-go based solution. My take on it: https://github.com/makasim/sentryhook

@dineshgowda24
Copy link

@evalphobia is this WIP?

@evalphobia
Copy link
Owner

@dineshgowda24

Sorry, I've not created yet.
I recently do not use sentry and logrus so much, so it's bit difficult to test and confirm PR and features on my environment.

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.

@acabarbaye
Copy link

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.

@junchaw
Copy link

junchaw commented May 2, 2024

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 sentry.Flush(5 * time.Second) ! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants