[avalanche] Add getter and setter RPCs to retrieve/set the staking reward winner
ClosedPublic
Actions

Authored by Fabien on Oct 18 2023, 18:43.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

Summary

Get, recompute and set the payout script from the previous block hash.

Depends on D14648.

Test Plan

./test/functional/test_runner.py abc_mining_stakingrewards

Repository

rABC Bitcoin ABC

Branch

set_staking_reward

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25370
Build 50321: Build Diff	lint-circular-dependencies · build-diff · build-debug · build-clang-tidy · build-without-wallet · build-clang
Build 50320: arc lint + arc unit

Fabien requested review of this revision.Oct 18 2023, 18:43

PiRK accepted this revision.Oct 19 2023, 07:31

PiRK added a subscriber: PiRK.

PiRK added inline comments.

src/rpc/avalanche.cpp
1145	Not something that needs to done in this diff, but I noticed that avalanche RPCs raise RPC_INVALID_PARAMETER in such a case while other RPCs raise RPC_INVALID_ADDRESS_OR_KEY. Maybe this whole process of taking the lock, looking up a block index and raising the RPC error or returning the CBlockIndex* would be worth factoring out in a helper function. It is repeated a lot.