StackExchange.Redis icon indicating copy to clipboard operation
StackExchange.Redis copied to clipboard

WIP: ILogger for ConnectionMultiplexer

Open NickCraver opened this issue 2 years ago • 0 comments

Working on ILogger support, but without new overloads possibly. Since we support Action<ConfigurationOptions> this can be added today e.g. with o => o.Logger = myLogger:

var muxer = ConnectionMultiplexer.Connect("localhost", o => o.Logger = myLogger);

...or we could add sibling overloads to the existing TextWriter. I'd love to not create a ^2 matrix every time we do this, though. Starting simpler for that reason. Note: this is on top of #2050 for simplicity, will roll against main when merged.

TODOs here:

  • [ ] General sanity check
  • [ ] Overloads decision
  • [ ] Tests of this implementation
  • [ ] Seeing which additional things need logging, we should make a list and check it here.

NickCraver avatar Mar 22 '22 02:03 NickCraver