go-goyave / goyave

🍐 The enterprise REST API framework
https://goyave.dev
MIT License
1.63k stars 68 forks source link

Add the ability to view performance data in Sentry #223

Open zKoz210 opened 3 months ago

zKoz210 commented 3 months ago

Proposal

Discussion:

There is a real lack of speed monitoring, such as Sentry.

Monitoring http speed, monitoring the speed of queries in the database, monitoring queries in the Redis storage (if there is such a thing in the future)

System-Glitch commented 3 months ago

There is a plugin for Datadog tracing support: dd-trace-goyave. I think it's definitely possible to do something similar for Sentry.

zKoz210 commented 3 months ago

https://github.com/getsentry/sentry-go https://docs.sentry.io/platforms/go/guides/http/configuration/options/

The official sentry package supports net/http. Is there some least costly way we can integrate this into your structure?

Thank you for answering all questions so quickly! I tried to write a small application with your framework and it really makes life easier in basic things.

System-Glitch commented 3 months ago

I had a look at their implementation. I think we can create an adapter for Goyave in a separate repository. It would require some work to make it very clean though. We need: