TelemetryConfiguration.Active

[idrivediesel2006]

According to Microsoft in this PR (microsoft/ApplicationInsights-dotnet#1148) they recommend using Telemetry.CreateDefault().

[nblumhardt]

Thanks for the heads-up. I think we should probably remove this example entirely - the only really useful way to configure this sink currently is with a connection string or via injection of the telemetry client through DI, IIRC 🤔

[Saibamen]

Please add (follow here):

[Obsolete("We do not recommend using TelemetryConfiguration.Active on .NET Core. See https://github.com/microsoft/ApplicationInsights-dotnet/issues/1152 for more details")]

here: https://github.com/serilog-contrib/serilog-sinks-applicationinsights/blob/dev/src/Serilog.Sinks.ApplicationInsights/LoggerConfigurationApplicationInsightsExtensions.cs#L65

And here: https://github.com/serilog-contrib/serilog-sinks-applicationinsights/blob/dev/src/Serilog.Sinks.ApplicationInsights/LoggerConfigurationApplicationInsightsExtensions.cs#L41

With #NETSTANDARD, like here: https://github.com/microsoft/ApplicationInsights-dotnet/blob/c9d420224a06d27ee74fba4b41cad7460bd63bd0/BASE/src/Microsoft.ApplicationInsights/TelemetryClient.cs#L40

[Saibamen]

Suggested change

	// TelemetryConfiguration.Active is obsolete
	// https://github.com/microsoft/ApplicationInsights-dotnet/pull/1148

No need to add this for readme

[Saibamen]

Suggested change

	// TelemetryConfiguration.Active is obsolete
	// https://github.com/microsoft/ApplicationInsights-dotnet/pull/1148

No need to add this for readme

[Saibamen]

I think we should probably remove this example entirely

@nblumhardt: I'm still using this style of configuring logger and output template.