Page MenuHomePhabricator

[avalanche] Deprecate the node availability score in the peerinfo RPC
ClosedPublic

Authored by Fabien on Tue, Feb 18, 15:39.

Details

Summary

This score is used internally by the node as a metric to filter out bad nodes in some cases. Unfortunately having it in the widely used peerinfo RPC caused the users to think it was a health metric for their node which it is not. Let's remove this field to avoid confusion, it was only added to help with testing.

Test Plan
ninja all check-all

Diff Detail

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

Event Timeline

Fabien requested review of this revision.Tue, Feb 18, 15:39
PiRK added a subscriber: PiRK.
PiRK added inline comments.
src/rpc/net.cpp
211–212 ↗(On Diff #52668)

layout

This revision is now accepted and ready to land.Tue, Feb 18, 17:12