bragma / winston-azure-application-insights

Azure Application Insights transport for Winston
MIT License
16 stars 26 forks source link

Incompatible with applicationinsights ^1.0.1: traces not sent to AI #8

Closed willmorgan closed 6 years ago

willmorgan commented 6 years ago

Hey @bragma, thanks for starting this library - it looks useful, and a colleague of mine has integrated it onto our project. We're currently using it inside a Function App deployment.

However, unfortunately, it seems that applicationinsights 1.x has broken this library's tracking. It also fails quietly:

ApplicationInsights:track() requires telemetry object and telemetryType to be specified. []

The AppInsights client trackTrace method now requires an object to be passed, rather than the old 3 parameter (msg, level, meta) signature:

It also looks like trackException has changed, too.

The migration guide is here: https://github.com/Microsoft/ApplicationInsights-node.js/tree/1.0.1#migrating-from-versions-prior-to-022

bragma commented 6 years ago

Thanks, please see #11