Differential D6914

[avalanche] Add buildavalanchproof RPC call
ClosedPublic
Actions

Authored by deadalnix on Jul 11 2020, 22:19.

Tags

None

Subscribers

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rABC454b88ccaea6: [avalanche] Add buildavalanchproof RPC call

Summary

As per title. This RPC allow to generate proof from a JSON description of it.

Test Plan

ninja all check-all

The integration test has been updated to use the RPC in question.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

avabuildproof

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 11989
Build 23792: Default Diff Build & Tests
Build 23791: Build without Wallet	build-without-wallet
Build 23790: Build with clang-10	build-clang-10
Build 23789: arc lint + arc unit

Event Timeline

deadalnix created this revision.Jul 11 2020, 22:19

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 11 2020, 22:19

deadalnix requested review of this revision.Jul 11 2020, 22:19

Harbormaster completed remote builds in B11980: Diff 22219.Jul 11 2020, 22:31

Fabien requested changes to this revision.Jul 13 2020, 07:44

Fabien added a subscriber: Fabien.

Fabien added inline comments.

src/rpc/avalanche.cpp
147 ↗	(On Diff #22219)	Why ?
168 ↗	(On Diff #22219)	Nit: since height = 0 is supposed to be the genesis block you can check for height < 1
194 ↗	(On Diff #22219)	Dito

This revision now requires changes to proceed.Jul 13 2020, 07:44

deadalnix added inline comments.Jul 13 2020, 15:48

src/rpc/avalanche.cpp
147 ↗	(On Diff #22219)	Leftover debug.
194 ↗	(On Diff #22219)	dito

Address comments

Harbormaster completed remote builds in B11989: Diff 22231.Jul 13 2020, 18:30

Fabien accepted this revision.Jul 13 2020, 18:31

This revision is now accepted and ready to land.Jul 13 2020, 18:31

Closed by commit rABC454b88ccaea6: [avalanche] Add buildavalanchproof RPC call (authored by deadalnix). · Explain WhyJul 13 2020, 19:39

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABC454b88ccaea6: [avalanche] Add buildavalanchproof RPC call.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

1 line

rpc/

141 lines

1 line

test/

functional/

abc_p2p_avalanche.py

22 lines

Diff 22231

src/avalanche/proof.h

Loading...

src/rpc/avalanche.cpp

Loading...

src/rpc/client.cpp

Loading...

test/functional/abc_p2p_avalanche.py

Loading...