search

huang-julien / nuxt-applicationinsights

ApplicationInsights module for Nuxt 3.
15 stars 0 forks source link

Quick Status Check #2

Closed MilanFox closed 7 months ago

MilanFox commented 7 months ago

Hey @huang-julien,

Thanks for your work here! I got here through an issue on the "old" Nuxt 2 solution. Just dropping a quick note to check in on the project's status. Is it considered finished, or are there ongoing updates in the pipeline?

I saw the blog post from Alex Lichter about how complex this is, so really trying not to sound pushy here ;) Just wanted to find out if I can use this already.

Thanks! Milan

huang-julien commented 7 months ago

Hey @MilanFox ! First thanks for looking at this package 😄 !

It is indeed still in progress, i was busy last few weeks but still plan on working on this package this week.

In the meantime, i'm trying to get in touch with microsoft to review this package. I'll work again on this package this week but I have a lot of custom implementation on a project at work so (as we stated with Alex), it kinda takes a lot of time to figure out how to optimize some things (and to make it generic enough). I plan to use this package next week to accelerate bug finding and improvements.

It should be fully working for applicaiton insight client which is basically just providing applicationinsight-web because everything works quite well. nuxt-applicationinsights exists mainly due to nitro-applicationinsights since applicationinsights-node is written in CJS and cannot auto-track with nitro/nuxt because it tries to patch native node modules...

I don't think this package needs more improvement aside from DX. Now most changes will come from nitro-applicationinsights since the server-side implementation is the hardest part (we have to manually track everything)

0.1 provides a very basic implementation of server side application insights + client application insights.

I think it SHOULD be usable but you will probably have some workaround to do. Feel free to open issues on this repo ! All suggestions are welcome !

MilanFox commented 7 months ago

Thanks for your reply, keep up the good work! I will keep an eye on this repo - no pressure though!

In the meantime MS seems to have published a new documentation on the OpenTelemetry API, so I will see if that gets me anywhere: https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-enable?tabs=nodejs

huang-julien commented 7 months ago

Thank you ! Yes i'm keeping an eye on OpenTelemetry, I'll probably migrate nitro-applicationinsights to it once applicationinsights reach 3.0.