microsoft / ApplicationInsights-dotnet-server

Microsoft Application Insights for .NET Web Applications
https://azure.microsoft.com/services/application-insights/
133 stars 67 forks source link

quick pulse endpoint config improvements. #1266

Closed TimothyMothra closed 5 years ago

TimothyMothra commented 5 years ago

QuickPulse Endpoint is set on the Module. The same value is internally set on the Processor, but only after the module starts collecting events. This makes it difficult to unit test because it's waiting for data to flow through these classes.

I made a change so that the 'Processor' endpoint is set during initialization methods. But that was still sensitive to which order the module and endpoint are initialized.

Summary

Note These are edge cases discovered while working on connection strings. Connection String support will come in the next PR.