Page MenuHomePhabricator

[avalanche] Skip processing compact proofs with no shortid
ClosedPublic

Authored by Fabien on Jun 28 2022, 18:54.

Details

Summary

A compact proofs message (avaproofs) with an empty shortid list is actually processed, accounted as a valid message and an empty request is sent in response. We can simplify this logic by skipping all this if there is nothing to process, not sending the empty request and not counting the message as a valid one. As a consequence this also improves the compact proof node counter logic (we are sure we received a useful compact proofs message vs any compact proofs message).

Depends on D11680.

Test Plan
ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable