Describe the bug
The command docs info for CLIENT KILL has an incorrect duplicate name between ADDR and LADDR. (And SKIPME is suspect.)
Details
Run command docs and inspect the details. In particular notice that ADDR and LADDR have the same name - ip:port. These names would be expected to be addr and laddr, but at minimum they must not be the same, because then parsing driven from this data can't tell the difference.
$11
client|kill
...
$9
arguments
..
$10
new-format
..
$4
name
$7
ip:port
$4
type
$6
string
$5
token
$4
ADDR
...
$4
name
$7
ip:port
$4
type
$6
string
$5
token
$5
LADDR
The other weird one is SKIPME use of name yes/no, instead of expected skipme:
name
$6
yes/no
$4
type
$6
string
$5
token
$6
SKIPME
Comment From: enjoy-binbin
what version are you using? i think the ip:port problem was fixed in #11051 (ohh, i see it is part of 7.2) and the client-kill problem was fixed in #11201 (and this one looks like it hasn't been released yet)
Comment From: jimsnab
$ redis-server --version
Redis server v=7.0.7 sha=00000000:0 malloc=jemalloc-5.2.1 bits=64 build=2260280010e18db8
Comment From: jimsnab
Agree, when those fixes are released, issue will be fixed. Thanks for finding!