rpc: Assert that passed arg names are equal to hardcoded ones
ClosedPublic
Actions

Authored by PiRK on Sep 1 2021, 11:52.

Details

Reviewers

majcosta

Group Reviewers

Restricted Project

Commits

rABCf726df976df3: rpc: Assert that passed arg names are equal to hardcoded ones

Summary

This is a backport of core#19386 [3/5]
https://github.com/bitcoin/bitcoin/pull/19386/commits/fa9708f94c01cb8bf2971bdf404af38c38fa341b

Depends on D10007

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 16508
Build 32878: Build Diff	lint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang · build-clang-tidy
Build 32877: arc lint + arc unit

Event Timeline

PiRK created this revision.Sep 1 2021, 11:52

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 1 2021, 11:52

PiRK requested review of this revision.Sep 1 2021, 11:52

teamcity edited the summary of this revision. (Show Details)Sep 1 2021, 11:53

PiRK added a child revision: D10009: rpc: Add option to hide RPCArg.Sep 1 2021, 11:54

Harbormaster completed remote builds in B16508: Diff 29721.Sep 1 2021, 12:15

majcosta accepted this revision.Sep 1 2021, 12:18

This revision is now accepted and ready to land.Sep 1 2021, 12:18

PiRK planned changes to this revision.Sep 1 2021, 13:56

PiRK requested review of this revision.Sep 9 2021, 08:31

This revision is now accepted and ready to land.Sep 9 2021, 08:31

Closed by commit rABCf726df976df3: rpc: Assert that passed arg names are equal to hardcoded ones (authored by MarcoFalke <falke.marco@gmail.com>, committed by PiRK). · Explain WhySep 10 2021, 06:59

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABCf726df976df3: rpc: Assert that passed arg names are equal to hardcoded ones.

Revision Contents
Changeset List

Path

Size

src/

rpc/

server.h

5 lines

Diff 29721

View Options

rpc: Assert that passed arg names are equal to hardcoded onesClosedPublicActions