logger.UpdateContext checks for disabled logger:
if l == disabledLogger {
return
}
I think this is problematic as it should check in a more general way, to handle also Nop and zero values. In my tests I found race conditions because I was reusing same global nop-logger. I think a better check would be something like it was done in #617. Do you agree? I can make a PR.
logger.UpdateContextchecks for disabled logger:I think this is problematic as it should check in a more general way, to handle also
Nopand zero values. In my tests I found race conditions because I was reusing same global nop-logger. I think a better check would be something like it was done in #617. Do you agree? I can make a PR.