[FEATURE] Add WithLogger extension method overloads to support non-generic ILogger

[tky753]

Problem

I am currently using a custom ILogger implementation. However, due to some specific internal constraints, the T type in my implementation is fixed and cannot be dynamically changed. This makes it impossible to adapt to ILogger directly.
Since ILogger inherits from ILogger, would it be possible to change the logger type across FusionCache from the specific ILogger to the more general ILogger?

Solution

Add extension function
public static IFusionCacheBuilder WithLogger(this IFusionCacheBuilder builder, Func<IServiceProvider, ILogger> factory)
and
public static IFusionCacheBuilder WithLogger(this IFusionCacheBuilder builder, ILogger logger)

[jodydonetti]

Hi @tky753 , to do that I'd have to change the internals of FusionCache and the main ctor.

It probably can be done without breaking everything, but I have to be very careful, mostly around overload resolution & the likes.

I'm in the process on doing a big overhaul of the logging part of FusionCache, and there are already quite a bit of tasks I'm looking at, so I've added this too to the list.

I'll post here a link to the issue with the proposed design as soon as I can.

Anyway, as a temporary solution, you can rely on the fact that ILogger<T> is just an empty marker interface for the plain ILogger interface, meaning id does not add anything.

Therefore, you can use something like this LoggerAdapter<T> implementation.

Hope this helps, let me know.