Page MenuHomePhabricator

[avalanche] Fix getdata log not showing stake contender inv type name
AcceptedPublic

Authored by roqqit on Fri, Nov 22, 18:30.

Details

Reviewers
PiRK
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

Turns out the "received getdata" log line only spits out hex if the inv type does not have an associated case in CInv::GetCommand(). This should be fixed for stake contenders because the hex is not only confusing but makes it more difficult to write tests for multiple inv types.

Test Plan
ninja check check-functional

Event Timeline

Owners added a reviewer: Restricted Owners Package.Fri, Nov 22, 18:30
roqqit requested review of this revision.Fri, Nov 22, 18:30
roqqit added a child revision: Restricted Differential Revision.Fri, Nov 22, 19:13
This revision is now accepted and ready to land.Fri, Nov 22, 20:46