Skip to content

Commit c85eb9c

Browse files
kwvgkwvg
committed
chore: add composite RPC bools to conversion table, deprecate old logic
We cannot simply get rid of `ParseBoolV` because upstream parsing logic is much more stricter, constituting a breaking change. This requires us to therefore follow the deprecation process.
1 parent 51eb7b6 commit c85eb9c

4 files changed

Lines changed: 52 additions & 22 deletions

File tree

‎src/rpc/client.cpp‎

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -257,6 +257,10 @@ static const CRPCConvertParam vRPCConvertParams[] =
257257
{ "submitchainlock", 2, "blockHeight" },
258258
{ "mnauth", 0, "nodeId" },
259259
// Compound RPCs (note: index position is offset by one to account for subcommand)
260+
{ "bls generate", 1, "legacy" },
261+
{ "bls fromsecret", 2, "legacy" },
262+
{ "coinjoinsalt generate", 1, "overwrite" },
263+
{ "coinjoinsalt set", 2, "overwrite" },
260264
{ "evodb repair", 1, "startBlock" },
261265
{ "evodb repair", 2, "stopBlock" },
262266
{ "evodb verify", 1, "startBlock" },
@@ -267,25 +271,34 @@ static const CRPCConvertParam vRPCConvertParams[] =
267271
{ "gobject prepare", 7, "outputIndex" },
268272
{ "gobject submit", 2, "revision" },
269273
{ "gobject submit", 3, "time" },
274+
{ "masternode connect", 2, "v2transport" },
270275
{ "masternode payments", 2, "count" },
271276
{ "protx diff", 1, "baseBlock" },
272277
{ "protx diff", 2, "block" },
278+
{ "protx diff", 3, "extended" },
279+
{ "protx list", 2, "detailed" },
273280
{ "protx list", 3, "height" },
274281
{ "protx listdiff", 1, "baseBlock" },
275282
{ "protx listdiff", 2, "block" },
276283
{ "protx register", 2, "collateralIndex" },
277284
{ "protx register", 3, "coreP2PAddrs", true },
285+
{ "protx register", 10, "submit" },
278286
{ "protx register_legacy", 2, "collateralIndex" },
279287
{ "protx register_legacy", 3, "coreP2PAddrs", true },
288+
{ "protx register_legacy", 10, "submit" },
280289
{ "protx register_evo", 2, "collateralIndex" },
281290
{ "protx register_evo", 3, "coreP2PAddrs", true },
282291
{ "protx register_evo", 10, "platformP2PAddrs", true },
283292
{ "protx register_evo", 11, "platformHTTPSAddrs", true },
293+
{ "protx register_evo", 13, "submit" },
284294
{ "protx register_fund", 2, "coreP2PAddrs", true },
295+
{ "protx register_fund", 9, "submit" },
285296
{ "protx register_fund_legacy", 2, "coreP2PAddrs", true },
297+
{ "protx register_fund_legacy", 9, "submit" },
286298
{ "protx register_fund_evo", 2, "coreP2PAddrs", true },
287299
{ "protx register_fund_evo", 9, "platformP2PAddrs", true },
288300
{ "protx register_fund_evo", 10, "platformHTTPSAddrs", true },
301+
{ "protx register_fund_evo", 12, "submit" },
289302
{ "protx register_prepare", 2, "collateralIndex" },
290303
{ "protx register_prepare", 3, "coreP2PAddrs", true },
291304
{ "protx register_prepare_legacy", 2, "collateralIndex" },
@@ -295,10 +308,15 @@ static const CRPCConvertParam vRPCConvertParams[] =
295308
{ "protx register_prepare_evo", 10, "platformP2PAddrs", true },
296309
{ "protx register_prepare_evo", 11, "platformHTTPSAddrs", true },
297310
{ "protx revoke", 3, "reason" },
311+
{ "protx revoke", 5, "submit" },
312+
{ "protx update_registrar", 6, "submit" },
313+
{ "protx update_registrar_legacy", 6, "submit" },
298314
{ "protx update_service", 2, "coreP2PAddrs", true },
315+
{ "protx update_service", 6, "submit" },
299316
{ "protx update_service_evo", 2, "coreP2PAddrs", true },
300317
{ "protx update_service_evo", 5, "platformP2PAddrs", true },
301318
{ "protx update_service_evo", 6, "platformHTTPSAddrs", true },
319+
{ "protx update_service_evo", 9, "submit" },
302320
{ "quorum dkgsimerror", 2, "rate" },
303321
{ "quorum dkgstatus", 1, "detail_level" },
304322
{ "quorum getdata", 1, "nodeId" },
@@ -307,12 +325,15 @@ static const CRPCConvertParam vRPCConvertParams[] =
307325
{ "quorum getrecsig", 1, "llmqType" },
308326
{ "quorum hasrecsig", 1, "llmqType" },
309327
{ "quorum info", 1, "llmqType" },
328+
{ "quorum info", 3, "includeSkShare" },
310329
{ "quorum isconflicting", 1, "llmqType" },
311330
{ "quorum listextended", 1, "height" },
312331
{ "quorum list", 1, "count" },
313332
{ "quorum memberof", 2, "scanQuorumsCount" },
333+
{ "quorum rotationinfo", 2, "extraShare" },
314334
{ "quorum selectquorum", 1, "llmqType" },
315335
{ "quorum sign", 1, "llmqType" },
336+
{ "quorum sign", 5, "submit" },
316337
{ "quorum verify", 1, "llmqType" },
317338
{ "quorum verify", 6, "signHeight" },
318339
};

‎src/rpc/util.cpp‎

Lines changed: 23 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
#include <script/signingprovider.h>
1313
#include <tinyformat.h>
1414
#include <util/check.h>
15+
#include <util/std23.h>
1516
#include <util/strencodings.h>
1617
#include <util/string.h>
1718
#include <util/system.h>
@@ -123,22 +124,30 @@ std::vector<unsigned char> ParseHexO(const UniValue& o, std::string strKey)
123124

124125
bool ParseBoolV(const UniValue& v, const std::string &strName)
125126
{
126-
std::string strBool;
127-
if (v.isBool())
128-
return v.get_bool();
129-
else if (v.isNum())
130-
strBool = ToString(v.getInt<int>());
131-
else if (v.isStr())
132-
strBool = v.get_str();
133-
134-
strBool = ToLower(strBool);
127+
if (std23::ranges::contains(gArgs.GetArgs("-deprecatedrpc"), "permissive_bool")) {
128+
std::string strBool;
129+
if (v.isBool())
130+
return v.get_bool();
131+
else if (v.isNum())
132+
strBool = ToString(v.getInt<int>());
133+
else if (v.isStr())
134+
strBool = v.get_str();
135+
136+
strBool = ToLower(strBool);
137+
138+
if (strBool == "true" || strBool == "yes" || strBool == "1") {
139+
return true;
140+
} else if (strBool == "false" || strBool == "no" || strBool == "0") {
141+
return false;
142+
}
143+
throw JSONRPCError(RPC_INVALID_PARAMETER, strprintf("%s must be true, false, yes, no, 1 or 0 (not '%s')", strName, strBool));
144+
}
135145

136-
if (strBool == "true" || strBool == "yes" || strBool == "1") {
137-
return true;
138-
} else if (strBool == "false" || strBool == "no" || strBool == "0") {
139-
return false;
146+
if (!v.isBool()) {
147+
throw JSONRPCError(RPC_INVALID_PARAMETER, strprintf("%s must be a JSON boolean. " /* Continued */
148+
"Pass -deprecatedrpc=permissive_bool to allow legacy boolean parsing.", strName));
140149
}
141-
throw JSONRPCError(RPC_INVALID_PARAMETER, strName+" must be true, false, yes, no, 1 or 0 (not '"+strBool+"')");
150+
return v.get_bool();
142151
}
143152

144153
namespace {

‎src/test/rpc_tests.cpp‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -529,12 +529,12 @@ BOOST_AUTO_TEST_CASE(rpc_bls)
529529
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls generate")));
530530
BOOST_CHECK_EQUAL(r.get_obj().find_value("scheme").get_str(), "basic");
531531

532-
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls generate 1")));
532+
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls generate true")));
533533
BOOST_CHECK_EQUAL(r.get_obj().find_value("scheme").get_str(), "legacy");
534534
std::string secret_legacy = r.get_obj().find_value("secret").get_str();
535535
std::string public_legacy = r.get_obj().find_value("public").get_str();
536536

537-
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls generate 0")));
537+
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls generate false")));
538538
BOOST_CHECK_EQUAL(r.get_obj().find_value("scheme").get_str(), "basic");
539539
std::string secret_basic = r.get_obj().find_value("secret").get_str();
540540
std::string public_basic = r.get_obj().find_value("public").get_str();
@@ -543,24 +543,24 @@ BOOST_AUTO_TEST_CASE(rpc_bls)
543543
BOOST_CHECK_EQUAL(r.get_obj().find_value("scheme").get_str(), "basic");
544544
BOOST_CHECK_EQUAL(r.get_obj().find_value("public").get_str(), public_basic);
545545

546-
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret_legacy + std::string(" 1")));
546+
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret_legacy + std::string(" true")));
547547
BOOST_CHECK_EQUAL(r.get_obj().find_value("scheme").get_str(), "legacy");
548548
BOOST_CHECK_EQUAL(r.get_obj().find_value("public").get_str(), public_legacy);
549549

550-
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret_basic + std::string(" 0")));
550+
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret_basic + std::string(" false")));
551551
BOOST_CHECK_EQUAL(r.get_obj().find_value("scheme").get_str(), "basic");
552552
BOOST_CHECK(r.get_obj().find_value("public").get_str() != public_legacy);
553553

554-
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret_basic + std::string(" 0")));
554+
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret_basic + std::string(" false")));
555555
BOOST_CHECK_EQUAL(r.get_obj().find_value("scheme").get_str(), "basic");
556556
BOOST_CHECK_EQUAL(r.get_obj().find_value("public").get_str(), public_basic);
557557

558-
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret_basic + std::string(" 1")));
558+
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret_basic + std::string(" true")));
559559
BOOST_CHECK_EQUAL(r.get_obj().find_value("scheme").get_str(), "legacy");
560560
BOOST_CHECK(r.get_obj().find_value("public").get_str() != public_basic);
561561

562562
std::string secret = "0b072b1b8b28335b0460aa695ee8ce1f60dc01e6eb12655ece2a877379dfdb51";
563-
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret + " 1"));
563+
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret + " true"));
564564
BOOST_CHECK_EQUAL(r.get_obj().find_value("scheme").get_str(), "legacy");
565565
BOOST_CHECK_EQUAL(r.get_obj().find_value("public").get_str(), "9379c28e0f50546906fe733f1222c8f7e39574d513790034f1fec1476286eb652a350c8c0e630cd2cc60d10c26d6f6ee");
566566
BOOST_CHECK_NO_THROW(r = CallRPC(std::string("bls fromsecret ") + secret));

‎test/functional/feature_llmq_connections.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ def get_mn_connection_count(self, node):
148148

149149
def get_mn_probe_count(self, node, q, check_peers):
150150
count = 0
151-
mnList = node.protx('list', 'registered', 1)
151+
mnList = node.protx('list', 'registered', True)
152152
peerList = node.getpeerinfo()
153153
mnMap = {}
154154
peerMap = {}

0 commit comments

Comments
 (0)