• Huang Zw's avatar
    Client tracking tracking-redir-broken push len is 2 not 3 (#8456) · f687ac0c
    Huang Zw authored
    When redis responds with tracking-redir-broken push message (RESP3),
    it was responding with a broken protocol: an array of 3 elements, but only
    pushes 2 elements.
    
    Some bugs in the test make this pass. Read the push reply
    will consume an extra reply, because the reply length is 3, but there
    are only two elements, so the next reply will be treated as third
    element. So the test is corrected too.
    
    Other changes:
    * checkPrefixCollisionsOrReply success should return 1 instead of -1,
      this bug didn't have any implications.
    * improve client tracking tests to validate more of the response it reads.
    f687ac0c
tracking.c 24.5 KB