Differential D11364

[avalanche] Ignore duplicated avahello messages
ClosedPublic
Actions

Authored by Fabien on Apr 15 2022, 09:30.

Tags

None

Subscribers

None

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rABC5c1189b9a2ae: [avalanche] Ignore duplicated avahello messages

Summary

And more generally don't allow a node to override its delegation without restarting. This is not a big deal because there is no such facility (yet) in the node, and it prevents a peer from sending messages repeatidly and use the node CPU for no reason.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Apr 15 2022, 09:30

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 15 2022, 09:30

Fabien requested review of this revision.Apr 15 2022, 09:30

Harbormaster completed remote builds in B18834: Diff 33231.Apr 15 2022, 09:45

PiRK accepted this revision.Apr 17 2022, 16:35

This revision is now accepted and ready to land.Apr 17 2022, 16:35

Closed by commit rABC5c1189b9a2ae: [avalanche] Ignore duplicated avahello messages (authored by Fabien). · Explain WhyApr 19 2022, 07:50

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC5c1189b9a2ae: [avalanche] Ignore duplicated avahello messages.

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

10 lines

test/

functional/

abc_p2p_avalanche_peer_discovery.py

6 lines

Diff 33250

src/net_processing.cpp

Loading...

test/functional/abc_p2p_avalanche_peer_discovery.py

Loading...