HomePhabricator

[avalanche] Fix bytes received tracking for avalanche messages

Description

[avalanche] Fix bytes received tracking for avalanche messages

Summary: getpeerinfo currently tracks all avalanche messages as "*other*" for bytesrecv_per_msg but correctly tracks all message types (avalanche included) for bytessent_per_msg. This fix adds them to bytesrecv_per_msg.

Test Plan:

ninja
bitcoind -daemon
bitcoin-cli getpeerinfo

Before patch example:

"bytesrecv_per_msg": {
  "*other*": 15359133,
  "addrv2": 533731,
  "cmpctblock": 501,
  "feefilter": 32,
  "getdata": 1220,
  "getheaders": 1053,
  "headers": 842001,
  "inv": 352088,
  "ping": 1338368,
  "pong": 1338304,
  "sendaddrv2": 24,
  "sendcmpct": 33,
  "sendheaders": 24,
  "tx": 726446,
  "verack": 24,
  "version": 146
},

After patch example:

"bytesrecv_per_msg": {
  "addrv2": 19592,
  "avahello": 351,
  "avaproof": 457,
  "feefilter": 32,
  "getheaders": 1053,
  "ping": 32,
  "pong": 32,
  "sendaddrv2": 24,
  "sendcmpct": 33,
  "verack": 24,
  "version": 146
},

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D16454

Details

Provenance
roqqitAuthored on Jul 11 2024, 22:55
roqqitPushed on Jul 12 2024, 15:54
Reviewer
Restricted Owners Package
Differential Revision
D16454: [avalanche] Fix bytes received tracking for avalanche messages
Parents
rABC020c3c10290b: net_processing: move avalanche related args to PeerManager::Options
Branches
Unknown
Tags
Unknown