InfrastructureExtensions GetService throws an error.

[qsdfplkj]

I use: _logger = this.GetService<ILogger<MyContext>>(); inside the constructor of MyContext

The call to InfrastructureExtensions.GetService however throws an error in case there is no logger registered.

I wasn't expecting this as i assumed it would behave like the regular serviceprovider. getservice.
as compared with GetRequiredService that could throw an error.

Is there any possibility to get an instance of the logger without getting the error in case it's not configured?
(I know i can add a parameter to the constructor but that isn't an option for some reason.) or maybe update the InfrastructureExtensions to support both getservice and getrequiredservice?

As a workaround I now use: new Logger<MyContext>(this.GetService<ILoggerFactory>());

[ajcvickers]

Note for triage: we should consider aligning here. Currently we throw the following, but I'm not sure how useful this is anymore:

Unhandled exception. System.InvalidOperationException:
Unable to resolve service for type 'Microsoft.Extensions.Logging.ILogger<SomeDbContext>'.
This is often because no database provider has been configured for this DbContext.
A provider can be configured by overriding the 'DbContext.OnConfiguring' method
or by using 'AddDbContext' on the application service provider. If 'AddDbContext' is used,
then also ensure that your DbContext type accepts a DbContextOptions<TContext> 
object in its constructor and passes it to the base constructor for DbContext.

@qsdfplkj You can do this instead:

var service = context.GetService<IDbContextOptions>()
    .Extensions.OfType<CoreOptionsExtension>().FirstOrDefault()
    ?.ApplicationServiceProvider
    ?.GetService<ILogger<SomeDbContext>>();

[ajcvickers]

Note from triage: we should align the behaviors here, but try to keep giving a useful exception for the case where no service provider has been registered, either by checking for this case specifically, or by changing the place where we call it internally.

[qsdfplkj]

Maybe just add a 'TryGetService' it will be clear for people using it. After all it is infrastructure.