Serilog / ILogger about lavspent.browserlogger

[petterlabraaten]

Thanks for the feedback!

I'm not experienced with Serilog, but I had a go tonight where I refactored away/split out anything related to .Net Cores ILogger and made a BrowserLogger independent of whether you choose the "native", Serilogs or other logging frameworks.

I'll commit some code after some rework/cleaning, but for Serilog's case the use would look something like this:

        public Startup(IConfiguration configuration)
        {
            //...

            Log.Logger = new LoggerConfiguration()
                 .Enrich.FromLogContext()
                 .WriteTo.BrowserLogger()
                 .CreateLogger();
        }

        public void ConfigureServices(IServiceCollection services)
        {
	    //...
            services.AddBrowserLogger();
            services.AddLogging(loggingBuilder =>
                     loggingBuilder.AddSerilog(dispose: true));
            services.Configure<BrowserLoggerOptions>(Configuration.GetSection("Logging"));
        }

        public void Configure(IApplicationBuilder app, IHostingEnvironment env)
        {
	    //...

            app.UseWebSockets();
            app.UseBrowserLoggerSerilog();

	    // This line has to be fixed, but for now...
            BrowserLoggerSink.Instance.Service = app.ApplicationServices.GetService<BrowserLoggerService>();
            app.UseMvc();
        }

Then from you code you either use the ILogger, or you can:

            Serilog.Log.Warning("This is a warning!");

Is this along the lines of what your want?

[emouawad]

Thanks a lot! didn't expect a fast one ❤️ ❤️ ❤️

It's definitely a step on the right track - would it much to ask if BrowserLogger can be configured from appsettings (https://github.com/serilog/serilog-settings-configuration) instead of: services.Configure(Configuration.GetSection("Logging"));
has it's own settings for serilog supports multiple sinks therefore we need multiple configurations:

Something like:

"WriteTo": [
{ "Name": "Console" },
{ "Name": "BrowserLoggerOptions", "Args": {
"ConsolePath": "console",
"WebConsole": {
"LogStreamUrl": "ws://localhost:5000/ls",
"ShowClassName": false
},
"LogLevel": {
"Default": "Information",
"System": "Information",
"Microsoft": "Information"
}
} }
],