microsoft / ApplicationInsights-node.js

Microsoft Application Insights SDK for Node.js
MIT License
320 stars 138 forks source link

[beta] ETA for when 3.0.0 will be out #1247

Closed lapa182 closed 2 months ago

lapa182 commented 7 months ago

Hello!

We are using the beta version of the SDK in our Next.js application but we would like to know if there's an ETA of when the plugin will be out of the beta version. Right now there are a lot of warnings that don't impact using it, but impact our logs in the server and the readability of it, so hence this question.

That's all right if there's no ETA, thanks for the hard work in any case!

hectorhdzg commented 7 months ago

@lapa182 we are targeting march of next year to have this out of beta. Can you elaborate on the warnings you are seeing?

lapa182 commented 7 months ago

Hi @hectorhdzg thanks for the reply! The warnings we are getting are the following:

ApplicationInsights:@azure/opentelemetry-instrumentation-azure-sdk [
  'Module @azure/core-tracing has been loaded before @azure/opentelemetry-instrumentation-azure-sdk so it might not work, please initialize it before requiring @azure/core-tracing'
]
ApplicationInsights:Invalid metric name: "\ASP.NET Applications(??APP_W3SVC_PROC??)\Request Execution Time". The metric name should be a ASCII string with a length no greater than 255 characters. []
ApplicationInsights:Invalid metric name: "\ASP.NET Applications(??APP_W3SVC_PROC??)\Requests/Sec". The metric name should be a ASCII string with a length no greater than 255 characters. []
ApplicationInsights:Invalid metric name: "\Process(??APP_WIN32_PROC??)\Private Bytes". The metric name should be a ASCII string with a length no greater than 255 characters. []
ApplicationInsights:Invalid metric name: "\Memory\Available Bytes". The metric name should be a ASCII string with a length no greater than 255 characters. []
ApplicationInsights:Invalid metric name: "\Processor(_Total)\% Processor Time". The metric name should be a ASCII string with a length no greater than 255 characters. []
ApplicationInsights:Invalid metric name: "\Process(??APP_WIN32_PROC??)\% Processor Time". The metric name should be a ASCII string with a length no greater than 255 characters. []
ApplicationInsights:Could not find ICACLS in expected location! This is necessary to use disk retry mode on Windows. []
ApplicationInsights:Sufficient file protection capabilities were not detected. Files will not be persisted []
ApplicationInsights:Sufficient file protection capabilities were not detected. Files will not be persisted []
ApplicationInsights:Sufficient file protection capabilities were not detected. Files will not be persisted []
ApplicationInsights:Sufficient file protection capabilities were not detected. Files will not be persisted []
ApplicationInsights:Sufficient file protection capabilities were not detected. Files will not be persisted []
ApplicationInsights:Sufficient file protection capabilities were not detected. Files will not be persisted []
ApplicationInsights:Sufficient file protection capabilities were not detected. Files will not be persisted []

We are using applicantioninsights-node.js@beta in a Next.js application.

hectorhdzg commented 6 months ago

Thanks @lapa182, sorry for late reply, metric name warnings should be gone now, the one related to file protection are interesting ones, are you running your program with some account that have limited permissions in the OS?

skhilliard commented 2 months ago

Any update as to when this version will come out of beta? Thanks!

lapa182 commented 2 months ago

Sorry for the late reply @hectorhdzg ,the warnings are gone. Quick question regarding beta, I've noticed it's out, because there's a difference in implementation but there's also a package called https://www.npmjs.com/package/@azure/monitor-opentelemetry, which one should we use with OpenTelemetry?

JacksonWeber commented 2 months ago

Closing this as Application Insights 3.0.0 SDK was released last week.