LoggerLookupRegistration fails if you have multiple instances of ILogger registered about rocklib.logging HOT 4 CLOSED

@RockLib/maintainers Take a look please

from rocklib.logging.

Looking now.... @bessdt do you have a code snippet that can be used to reproduce the issue?

from rocklib.logging.

@JasonBock You can reproduce it by adding another ILogger to the service container after calling AddLogger(). An example using the WebApplicationFactory from the the ASP.NET Testing Framework (Microsoft.AspNetCore.Mvc.Testing):

In Program.cs:

builder.Services.AddLogger();
builder.Logging.AddRockLibLoggerProvider();

In test:

var mockLogger = new Mock<ILogger>();
var app = new WebApplicationFactory<Program>().WithWebHostBuilder(builder =>
{
builder.ConfigureTestServices(services => { services.AddSingleton<ILogger>(mockLogger.Object); });
});

// Resolve ILogger from IOC Container to reproduce runtime exception
app.Services.GetRequiredService<ILogger>();

from rocklib.logging.

@devonbessemer, RockLib.Logging 4.0.2 package has been released to address this bug.

from rocklib.logging.