Page MenuHomePhabricator

[avalanche] Don't download proofs during IBD
ClosedPublic

Authored by Fabien on Jul 29 2022, 14:22.

Details

Reviewers
sdulfari
Group Reviewers
Restricted Project
Commits
rABC3501c57cc5f6: [avalanche] Don't download proofs during IBD
Summary

We already avoid requesting compact proofs, but this extends to relayed proofs as well.
This diffs prevents the node from requesting proofs when an inv is received during IBD, and also discards the proofs received via the network in the event a peer sent us spontaneously a proof.

Depends on D11815.

Test Plan
ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Branch
avalanche_dont_receive_proofs_ibd
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 19752
Build 39223: Build Difflint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 39222: arc lint + arc unit

Event Timeline

Fabien requested review of this revision.Jul 29 2022, 14:22
sdulfari requested changes to this revision.Jul 29 2022, 18:41
sdulfari added a subscriber: sdulfari.
sdulfari added inline comments.
test/functional/abc_p2p_proof_inventory.py
218

is there a reason to not restart all of the nodes?

This revision now requires changes to proceed.Jul 29 2022, 18:41

Restart all nodes for clarity

This revision is now accepted and ready to land.Jul 31 2022, 05:52
This revision was automatically updated to reflect the committed changes.