Lev Zakharov
Lev Zakharov
Hi! Any updates on this issue? Got the same panic in our production environment. **Details** go version: `1.22.0` zap version: `v1.25.0` ``` panic({0x1d3e9e0?, 0x34e37e0?}) runtime/panic.go:914 +0x21f go.uber.org/zap/zapcore.(*jsonEncoder).safeAddString(0xc0376cd500, {0x0, 0x5c}) go.uber.org/[email protected]/zapcore/json_encoder.go:490...
Okay, we're trying to reproduce this case locally. Also, as I see, the implementation of `safeAddString` has already changed in `v1.26.0`. Perhaps this problem is no longer relevant for new...
Got the same `runtime error: invalid memory address or nil pointer dereference` with `v1.27.0`: ``` panic({0x1de8420?, 0x36187d0?}) runtime/panic.go:914 +0x21f go.uber.org/zap/zapcore.safeAppendStringLike[...](0x227cd38?, 0x227cdc0, 0xc00fd719e0, {0x0, 0x5c}) go.uber.org/[email protected]/zapcore/json_encoder.go:527 +0x54 go.uber.org/zap/zapcore.(*jsonEncoder).safeAddString(...) go.uber.org/[email protected]/zapcore/json_encoder.go:489 go.uber.org/zap/zapcore.(*jsonEncoder).AppendString(0xc003f8d500, {0x0,...
@sethvargo sorry for disturbing, can you review please? This PR will help us a lot. Thanks in advance!
Tests fixed in #2808.
@ofekshenawa sorry for disturbing, can you review please? This PR can help a lot. Thanks in advance!
Another option is to make the [baseClient.onClose](https://github.com/redis/go-redis/blob/21bd40a47e56e61c0598ea1bdf8e02e67d1aa651/redis.go#L189) configurable.
Hello, thank you for the issue. > Columns are not aligned. Both, center and right alignment are not working. What do you mean by that? As I can see, output...
Sorry, but this it not the right output: 1. column `a` should be centered due to `-c 0`; 2. values in `b` and `d` should not be aligned with spaces....
There is no need to align columns with spaces - alignment must be done when rendering the markdown.