search

evalphobia / logrus_sentry

sentry hook for logrus
MIT License
194 stars 78 forks source link

Support sentry-go? #85

Open danqing opened 5 years ago

danqing commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

That'd be cool.

I've also posted in the sentry-go repo (https://github.com/getsentry/sentry-go/issues/43) with some basic code I'm using right now, fwiw.

makasim commented 4 years ago

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

dineshgowda24 commented 3 years ago

@evalphobia is this WIP?

evalphobia commented 3 years ago

@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 commented 2 years ago

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 commented 6 months ago

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) ! :)