ApplicationInsights-node.js icon indicating copy to clipboard operation
ApplicationInsights-node.js copied to clipboard

Support for tedious (MS-SQL) and Pino?

Open DevRCRun opened this issue 2 years ago • 2 comments

Thanks for all your work on the new version. I see the following list designated for bundled support in the preview / beta 👍

Distributed Tracing HTTP/HTTPS MongoDB MySQL Postgres Redis Redis-4 Azure SDK

I'm a little hazy on what is necessary to support MS-SQL (tedious) and Pino, which look to be available in open telemetry. Will it be enough to add a TraceHandler in ApplicationInsightsClient for @opentelemetry/instrumentation-pino and @opentelemetry/instrumentation-tedious per the new readme or does this come with caveats?

DevRCRun avatar Feb 16 '23 14:02 DevRCRun

@DevRCRun you can take a look at the sample code we have to add ExpressInstrumentation here, you can add any other OpenTelemetry Instrumentation the same way

hectorhdzg avatar Feb 16 '23 17:02 hectorhdzg

Be aware though that @opentelemetry/instrumentation-pino only adds a few tracing specific records to the pino record - it does not help you in sending logs created through the pino logging interface to application insights, like the built-in log instrumentations for console, winston and bunyan would do.

See also https://github.com/microsoft/node-diagnostic-channel/issues/118#issuecomment-1600730013

dmunch avatar Jun 21 '23 12:06 dmunch