graphql-platform icon indicating copy to clipboard operation
graphql-platform copied to clipboard

Application insights support v11

Open woeterman94 opened this issue 3 years ago • 5 comments

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

woeterman94 avatar Aug 24 '21 14:08 woeterman94

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

michaelstaib avatar Aug 25 '21 16:08 michaelstaib

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar May 04 '22 11:05 stale[bot]

Well this issue is still in "backlog"

woeterman94 avatar May 04 '22 12:05 woeterman94

@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

Ricky-G avatar May 10 '22 09:05 Ricky-G

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 avatar May 10 '22 09:05 PascalSenn

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

PascalSenn avatar Sep 07 '22 15:09 PascalSenn