[avalanche] Add a RPC to check if a block/tx is finalized by avalanche
ClosedPublic
Actions

Authored by Fabien on Sep 15 2022, 15:31.

Details

Reviewers

sdulfari

Group Reviewers

Restricted Project

Commits

Summary

This will make it easier for exchanges to implement 1-block confirmation.

Test Plan

With clang and debug:

ninja all check-all

Run the ASAN and TSAN builds.

Repository

rABC Bitcoin ABC

Branch

avalanche_is_finalized

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 20052
Build 39792: Build Diff	lint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 39791: arc lint + arc unit

Fabien requested review of this revision.Sep 15 2022, 15:31

sdulfari requested changes to this revision.Sep 15 2022, 15:47

sdulfari added a subscriber: sdulfari.

sdulfari added inline comments.

src/rpc/avalanche.cpp
1042 ↗	(On Diff #34989)	Examples are missing <blockhash>
1055 ↗	(On Diff #34989)	blockhash isn't optional, but the tests don't test for the case where this is null
1072 ↗	(On Diff #34989)	The mempool check isn't needed if blockhash is required.

This revision now requires changes to proceed.Sep 15 2022, 15:47

Add txindex support, add a few safety null check

sdulfari requested changes to this revision.Sep 15 2022, 16:45

sdulfari added inline comments.

src/validation.cpp
3492 ↗	(On Diff #34990)	needs to exit early on nullptr
test/functional/abc_rpc_isfinal.py
108 ↗	(On Diff #34990)	nit: from here down, "mempool_txid" isn't in the mempool, so the naming is confusing