Log DI container creation exceptions

[drwatson1]

Log DI container creation exceptions

[tsvx]

It has turned out that this approach is incomplete. We should dispose the logger, it turns out this is given in the documentation.

[drwatson1]

You are right, thank you for pointing it out. I've decided to improve logging in the template based on the latest versions of Serilog extensions and documentation.

[drwatson1]

I've updated the template just now - https://marketplace.visualstudio.com/items?itemName=sergey-tregub.asp-net-core-restful-service-template.

Is the solution suitable for you?
Is there something else that I can improve?

[tsvx]

Thank you, Sergey. Looks like what I wanted. I am definitely going to test this solution when I start a new ASP.NET Core project.

One note on the cafec9d commit, it seems that the ConfigureLogging(..) call is redundant after adding UseSerilog(..). Also, that call replaces Log.Logger which is supposed to be modified (or replaced) in the UseSerilog(..) call. Please correct me if I am wrong.

[drwatson1]

You are right as always :) I forgot to remove ConfigureLogging after UseSerilog was added.

[drwatson1]

I close the issue now. Feel free to open a new issue if you find any other problems. Thank you for your contribution.