-
Notifications
You must be signed in to change notification settings - Fork 358
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Load ITelemetryProcessors when setting up the TelemetryConfigration #2657
Conversation
Hello, Any chance to have this fix merged ? |
@luthus -- I know this is over a year old. Are you still available to push this through? If not, I can add a simple unit test here and get it in for you. |
Pinging @cijothomas here as well for another set of eyes. |
{ | ||
foreach (ITelemetryProcessorFactory telemetryProcessorFactory in telemetryProcessorFactories) | ||
{ | ||
configuration.TelemetryProcessorChainBuilder.Use(telemetryProcessorFactory.Create); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ApplicationInsights adds processors to default sink. Not sure if its applicable here.
This approach won’t work as the ITelemeteryProcessorFactory is in Microsoft.ApplicationInsights.AspNetCore and this is marked as private in runtimeAssemblies. Will have to think of another approach to fix this. |
Should fix Azure/azure-functions-host#3741