support for pino.js logger

[vitoladev]

It would be awesome to support pino.js Logger, it currently has 3M weekly downloads on npm, great project.

[hectorhdzg]

@vitoladev we will using OpenTelemetry in Application Insights SDKs, pino is currently supported in OpenTelemetry more info here, we have a beta version of Application Insights using OpenTelemetry that you could start using along with pino instrumentation, more info about beta here

[dmunch]

@hectorhdzg I've had a look into using pino.js with the instrumentation you've linked above, and I think there's a bit of a misunderstanding of what the pino instrumentation is doing.

From reading through the source, I've understood now that the instrumentation is only adding some additional fields into the log record (trace_id, span_id, trace_flags) - however, it is not automatically sending logs created through the pino API to the diagnostic channel, like I would have hoped for.

[hectorhdzg]

@dmunch yes currently only extra tracing properties are added to the logs, Logs SDK was recently added in OpenTelemetry JS, that is why auto generation of logs for this kind of libraries is still not available, this kind of functionality would be added soon.