BUG/MEDIUM: peers: stop considering ack messages teaching a full resync
authorEmeric Brun <ebrun@haproxy.com>
Thu, 4 Mar 2021 09:27:10 +0000 (10:27 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Thu, 29 Apr 2021 09:12:18 +0000 (11:12 +0200)
commitaceed04e0ebb7b0b21ae54092fc15ebf379fe69d
tree9b4bd5e649ca155b2c68c4f9c9c502f7f6eb73fd
parentfcd6f73f0e6cb3a026c0ffa4b65a138b1586e330
BUG/MEDIUM: peers: stop considering ack messages teaching a full resync

The re-con cursor was updated receiving any ack message
even if we are pushing a complete resync to a peer. This cursor
is reset at the end of the resync but if the connection is broken
during resync, we could re-start at an unwanted point.

With this patch, the peer stops to consider ack messages pushing
a resync since the resync process has is own acknowlegement and
is always restarted from the beginning in case of broken connection.

This patch should be backported on all supported branches ( >= 1.6 )

(cherry picked from commit b0d60bed36cfbfead5a35e6a9520e8d5e9345a7f)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 39555d62ef6a7cec5c6c1bf20bd2d53cb45555fd)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit b8497151da9ebc5764e039aa1cd904497d1b6328)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
src/peers.c