sentry-go
sentry-go copied to clipboard
getsentry
Add Fiber integration
@arsmn thanks for the contribution. Sorry we haven't had the bandwidth to review this yet. On a high level I expect this is pretty much the same as the fasthttp integration with types adapted to match fiber's.
I'll have a look at this and Buffalo (https://github.com/getsentry/sentry-go/pull/196) in the coming weeks, perhaps start moving integrations off into their own modules to avoid an ever increasing list of deps in the main go.mod file.
Would be nice if this was merged in with official support for Fiber and Sentry. Are there any issues @rhcarvalho?
@arsmn thanks for the contribution. Sorry we haven't had the bandwidth to review this yet. On a high level I expect this is pretty much the same as the fasthttp integration with types adapted to match fiber's.
I'll have a look at this and Buffalo (#196) in the coming weeks, perhaps start moving integrations off into their own modules to avoid an ever increasing list of deps in the main go.mod file.
This is open for 1.5 years now. Is this still on the road map?
I think @arsmn should create PR under the https://github.com/gofiber/contrib.
Fiber integration is now available https://github.com/gofiber/contrib/tree/main/fibersentry Thank you all
Hey @cleptric & @arsmn any chance we could get this merged in? I'd love to have Sentry officially support this!
@robd003 Adding a Fiber integration is definitely on our road map, but I can't provide you with an ETA when this may land.
