Differential D10670

[avalanche] Introduce a conflicting proof cooldown
AbandonedPublic
Actions

Authored by Fabien on Dec 14 2021, 15:51.

Tags

None

Subscribers

None

Details

Reviewers

None

Group Reviewers

Restricted Project

Maniphest Tasks

Restricted Maniphest Task

Summary

This will reject any proof that conflict with a peer which has been registered in under 1h (default value). This rate limit will prevent an attacker from impacting the network by constantly replacing a peer, or generating proof polls to solve the conflict.

Ref T1854.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

avalanche_conflicting_proof_cooldown

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 17627
Build 35080: Build Diff	build-without-wallet · build-clang · build-diff · build-clang-tidy · build-debug · lint-circular-dependencies
Build 35079: arc lint + arc unit

Event Timeline

Fabien created this revision.Dec 14 2021, 15:51

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 14 2021, 15:51

Fabien requested review of this revision.Dec 14 2021, 15:51

Fabien mentioned this in D10525: [avalanche] Make it possible to replace a proof.Dec 14 2021, 15:54

Fabien added a child revision: D10525: [avalanche] Make it possible to replace a proof.Dec 14 2021, 15:55

Harbormaster completed remote builds in B17627: Diff 31391.Dec 14 2021, 16:05

Fabien planned changes to this revision.Dec 15 2021, 16:22

Fabien removed a child revision: D10525: [avalanche] Make it possible to replace a proof.Dec 22 2021, 15:18

Fabien abandoned this revision.Feb 9 2022, 13:32

Revision Contents
Changeset List

Path

Size

src/

avalanche/

7 lines

peermanager.cpp

28 lines

test/

peermanager_tests.cpp

121 lines

6 lines

Diff 31391

src/avalanche/avalanche.h

Loading...

src/avalanche/peermanager.cpp

Loading...

src/avalanche/test/peermanager_tests.cpp

Loading...

src/init.cpp

Loading...