mondago / laravel-azure-application-insights

Enable Azure Application Insights in your Laravel project
MIT License
5 stars 6 forks source link

`trackEvent` should not call `flush` #14

Open ingalless opened 1 year ago

ingalless commented 1 year ago

Flushing waits for the queue to be sent to app insights. It makes sense for "TrackRequest" to flush, as this is one of the last things that happens. However, throughout a request we may wish to add events to the queue. Therefore, we shouldn't flush.