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>
Wed, 28 Apr 2021 12:56:33 +0000 (14:56 +0200)
commit39555d62ef6a7cec5c6c1bf20bd2d53cb45555fd
tree64415e8c793d0566855490117146f4c3a9a8956e
parenteb2f8e0c27308c6a56f59ff00c2a2232e980ebca
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>
src/peers.c