dbft: prohibit WatchOnly to send RecoveryMessage

Fix a bug introduced by ffa8c1f. Signed-off-by: Anna Shaleva <shaleva.ann@nspcc.ru>
nspcc-dev · Nov 28, 2024 · 2faf31a · 2faf31a
1 parent 82ad778
commit 2faf31a
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/dbft.go b/dbft.go
@@ -610,6 +610,11 @@ func (d *DBFT[H]) onCommit(msg ConsensusPayload[H]) {
 }
 
 func (d *DBFT[H]) onRecoveryRequest(msg ConsensusPayload[H]) {
+	// Only validators are allowed to send consensus messages.
+	if d.Context.WatchOnly() {
+		return
+	}
+
 	if !d.CommitSent() && (!d.isAntiMEVExtensionEnabled() || !d.PreCommitSent()) {
 		// Ignore the message if our index is not in F+1 range of the
 		// next (%N) ones from the sender. This limits recovery