search

ChilliCream / graphql-platform

Welcome to the home of the Hot Chocolate GraphQL server for .NET, the Strawberry Shake GraphQL client for .NET and Banana Cake Pop the awesome Monaco based GraphQL IDE.
https://chillicream.com
MIT License
5.24k stars 744 forks source link

Application insights support v11 #4122

Open woeterman94 opened 3 years ago

woeterman94 commented 3 years ago

Is your feature request related to a problem?

See https://github.com/ChilliCream/hotchocolate/issues/1596

The solution you'd like

Is there any way we can integrate / hook on application insights with Hotchocolate 11?

The docs provided here: https://github.com/ChilliCream/hotchocolate/issues/1596 seem outdated. (https://github.com/ChilliCream/hotchocolate/blob/develop/src/HotChocolate/Core/src/Execution/Instrumentation/DiagnosticEventListener.cs)

Product

Hot Chocolate

michaelstaib commented 3 years ago

Yes... its simpler today... you can just create a listener and inject the telemetry client in there

woeterman94 commented 2 years ago

Well this issue is still in "backlog"

Ricky-G commented 2 years ago

@woeterman94 I had to do this recently and ended up with something like here https://stackoverflow.com/questions/70167150/azure-application-insights-integration-with-graphql-api-hotchocolate-framework

Ideally the most common integrations like Elastic, app insights etc should be out of the box, will if I can do a PR

PascalSenn commented 2 years ago

We have a open telemetry integration. I guess this is the direction most of the tracing providers go. Cant you just plug this into Azure Monitor?

PascalSenn commented 2 years ago

@woeterman94 did you see https://docs.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-overview