backblaze icon indicating copy to clipboard operation
backblaze copied to clipboard

Fix exception when using default logger value for BackblazeClient constructors

Open logiclrd opened this issue 1 year ago • 1 comments

The BackblazeClient class has a few constructor overloads that take a bunch of parameters explicitly, including an ILoggerFactory. The default value for these is null. This was causing an exception deeper down because the base class Storage assumes it always has a valid ILoggerFactory. This PR fixes the issue by making Storage substitute NullLoggerFactory.Instance when it receives null.

logiclrd avatar Oct 08 '24 07:10 logiclrd

Added regression tests. :-)

logiclrd avatar Oct 10 '24 16:10 logiclrd