Umbraco 10 Application insights with Serilog sink

Program.cs

using Microsoft.ApplicationInsights.Extensibility;
using Serilog;

public class Program
{
	public static void Main(string[] args)
		=> CreateHostBuilder(args)
			.Build()
			.Run();

	public static IHostBuilder CreateHostBuilder(string[] args) =>
		Host.CreateDefaultBuilder(args)
			.ConfigureUmbracoDefaults()
      //Boot time logging before Umbraco sets up SeriLog, optional.
			.UseSerilog((context, services, configuration) =>
			{
				configuration
				.ReadFrom.Configuration(context.Configuration)
				.ReadFrom.Services(services)
				.Enrich.FromLogContext().WriteTo.ApplicationInsights(
						services.GetRequiredService<TelemetryConfiguration>()
						, TelemetryConverter.Traces);				
			})
			.ConfigureWebHostDefaults(webBuilder =>
			{
				webBuilder.UseStaticWebAssets();
				webBuilder.UseStartup<Startup>();
			});
}

Startup.cs

public void ConfigureServices(IServiceCollection services)
{
		services.AddApplicationInsightsTelemetry();
    
    //AddSingleton - I have no idea if this is correct or not.
    //Registering a ILogEventSink before Umbraco setsup Serilog means the ReadFrom.Services(services) call picks up Application Insights.
    
		services.AddSingleton<ILogEventSink>(sp => new ApplicationInsightsSink(sp.GetRequiredService<TelemetryClient>(), TelemetryConverter.Traces));

    services.AddUmbraco(_env, _config)
      ....
}