microsoft / ApplicationInsights-node.js

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

Elaborate fix for #1226 #1242

Closed datenreisender closed 6 months ago

datenreisender commented 7 months ago

A more elaborate fix for trying to load the default configuration during runtime which only exists in the source code. That loading previously failed when the code is bundled. So now the default configuration is already loaded when bundling the code.

This is a more elaborate fix than #1241. I would prefer this solution but I do not know which style you prefer.

Fixes #1226.

datenreisender commented 7 months ago

@microsoft-github-policy-service agree

datenreisender commented 6 months ago

Closing this, since this is now fixed by #1241.