From 851c5a632dc3869a0c415c62a2a8bde40cfe0818 Mon Sep 17 00:00:00 2001 From: Konstantin Akimov Date: Sun, 7 Jul 2024 17:03:02 +0700 Subject: [PATCH] feat: mark feature rpc filter by "platform-user" deprecated --- src/init.cpp | 1 + src/rpc/server.cpp | 6 +- ...r.py => rpc_deprecated_platform_filter.py} | 21 +---- test/functional/rpc_external_queue.py | 92 +++++++++++++++++++ test/functional/test_runner.py | 3 +- 5 files changed, 102 insertions(+), 21 deletions(-) rename test/functional/{rpc_platform_filter.py => rpc_deprecated_platform_filter.py} (82%) create mode 100755 test/functional/rpc_external_queue.py diff --git a/src/init.cpp b/src/init.cpp index 005cca71c5fa8..3dbab5a09888d 100644 --- a/src/init.cpp +++ b/src/init.cpp @@ -746,6 +746,7 @@ void SetupServerArgs(NodeContext& node) argsman.AddArg("-llmq-qvvec-sync=:", strprintf("Defines from which LLMQ type the masternode should sync quorum verification vectors. Can be used multiple times with different LLMQ types. : %d (sync always from all quorums of the type defined by ), %d (sync from all quorums of the type defined by if a member of any of the quorums)", (int32_t)llmq::QvvecSyncMode::Always, (int32_t)llmq::QvvecSyncMode::OnlyIfTypeMember), ArgsManager::ALLOW_ANY, OptionsCategory::MASTERNODE); argsman.AddArg("-masternodeblsprivkey=", "Set the masternode BLS private key and enable the client to act as a masternode", ArgsManager::ALLOW_ANY | ArgsManager::SENSITIVE, OptionsCategory::MASTERNODE); argsman.AddArg("-platform-user=", "Set the username for the \"platform user\", a restricted user intended to be used by Dash Platform, to the specified username.", ArgsManager::ALLOW_ANY, OptionsCategory::MASTERNODE); + argsman.AddArg("-deprecated-platform-user", "Let use -platform-user since it is deprecated from v21", ArgsManager::ALLOW_ANY, OptionsCategory::MASTERNODE); argsman.AddArg("-acceptnonstdtxn", strprintf("Relay and mine \"non-standard\" transactions (%sdefault: %u)", "testnet/regtest only; ", !testnetChainParams->RequireStandard()), ArgsManager::ALLOW_ANY | ArgsManager::DEBUG_ONLY, OptionsCategory::NODE_RELAY); argsman.AddArg("-dustrelayfee=", strprintf("Fee rate (in %s/kB) used to define dust, the value of an output such that it will cost more than its value in fees at this fee rate to spend it. (default: %s)", CURRENCY_UNIT, FormatMoney(DUST_RELAY_TX_FEE)), ArgsManager::ALLOW_ANY | ArgsManager::DEBUG_ONLY, OptionsCategory::NODE_RELAY); diff --git a/src/rpc/server.cpp b/src/rpc/server.cpp index a47772a689557..513c8d6fb09e5 100644 --- a/src/rpc/server.cpp +++ b/src/rpc/server.cpp @@ -536,8 +536,12 @@ UniValue CRPCTable::execute(const JSONRPCRequest &request) const static bool ExecuteCommand(const CRPCCommand& command, const JSONRPCRequest& request, UniValue& result, bool last_handler, const std::multimap>& mapPlatformRestrictions) { const NodeContext& node = EnsureAnyNodeContext(request.context); + // TODO: remove -platform-user completely + if (gArgs.IsArgSet("-platform-user") && !gArgs.IsArgSet("-deprecated-platform-user")) { + throw JSONRPCError(RPC_PLATFORM_RESTRICTION, "-platform-user is deprecated. Use instead whitelists. If you still need to use -platform-user by any reason, add one more argument -deprecated-platform-user"); + } // Before executing the RPC Command, filter commands from platform rpc user - if (node.mn_activeman && request.authUser == gArgs.GetArg("-platform-user", defaultPlatformUser)) { + if (node.mn_activeman && gArgs.IsArgSet("-platform-user") && request.authUser == gArgs.GetArg("-platform-user", defaultPlatformUser)) { // replace this with structured binding in c++20 std::string command_name = command.name; if (!command.subname.empty()) command_name += " " + command.subname; diff --git a/test/functional/rpc_platform_filter.py b/test/functional/rpc_deprecated_platform_filter.py similarity index 82% rename from test/functional/rpc_platform_filter.py rename to test/functional/rpc_deprecated_platform_filter.py index 200ed96f48d15..2a4836e8ea279 100755 --- a/test/functional/rpc_platform_filter.py +++ b/test/functional/rpc_deprecated_platform_filter.py @@ -38,6 +38,8 @@ def setup_chain(self): f.write(masternodeblskey+"\n") f.write(rpcauthplatform+"\n") f.write(rpcauthoperator+"\n") + f.write("platform-user=platform-user\n") + f.write("deprecated-platform-user=true\n") def run_test(self): url = urllib.parse.urlparse(self.nodes[0].url) @@ -114,24 +116,5 @@ def test_command(method, params, auth, expected_status, should_not_match=False): test_command("debug", ["1"], rpcuser_authpair_operator, 200) - self.log.info("Restart node with -rpcexternaluser") - self.restart_node(0, extra_args=["-rpcexternaluser=platform-user"]) - - external_log_str = "HTTP: Calling handler for external user" - expected_log_str = "ThreadRPCServer method=" - with self.nodes[0].assert_debug_log(expected_msgs=[expected_log_str, external_log_str]): - test_command("getbestblockhash", [], rpcuser_authpair_platform, 200) - with self.nodes[0].assert_debug_log(expected_msgs=[expected_log_str], unexpected_msgs = [external_log_str]): - test_command("getbestblockhash", [], rpcuser_authpair_operator, 200) - - self.log.info("Restart node with multiple external users") - self.restart_node(0, extra_args=["-rpcexternaluser=platform-user,operator"]) - with self.nodes[0].assert_debug_log(expected_msgs=[expected_log_str, external_log_str]): - test_command("getbestblockhash", [], rpcuser_authpair_platform, 200) - with self.nodes[0].assert_debug_log(expected_msgs=[expected_log_str, external_log_str]): - test_command("getbestblockhash", [], rpcuser_authpair_operator, 200) - - - if __name__ == '__main__': HTTPBasicsTest().main() diff --git a/test/functional/rpc_external_queue.py b/test/functional/rpc_external_queue.py new file mode 100755 index 0000000000000..750084bf0d4b8 --- /dev/null +++ b/test/functional/rpc_external_queue.py @@ -0,0 +1,92 @@ +#!/usr/bin/env python3 +# Copyright (c) 2024 The Dash Core developers +# Distributed under the MIT software license, see the accompanying +# file COPYING or http://www.opensource.org/licenses/mit-license.php. +"""Test that commands submitted by the platform user are filtered.""" + +from test_framework.test_framework import BitcoinTestFramework +from test_framework.util import str_to_b64str, assert_equal + +import http.client +import json +import os +import urllib.parse + + +class HTTPBasicsTest(BitcoinTestFramework): + def set_test_params(self): + self.num_nodes = 1 + self.supports_cli = False + + def setup_nodes(self): + self.add_nodes(self.num_nodes) + self.start_nodes() + + def setup_chain(self): + super().setup_chain() + # Append rpcauth to dash.conf before initialization + rpcauthplatform = "rpcauth=platform-user:dd88fd676186f48553775d6fb5a2d344$bc1f7898698ead19c6ec7ff47055622dd7101478f1ff6444103d3dc03cd77c13" + # rpcuser : platform-user + # rpcpassword : password123 + rpcauthoperator = "rpcauth=operator:e9b45dd0b61a7be72155535435365a3a$8fb7470bc6f74d8ceaf9a23f49b06127723bd563b3ed5d9cea776ef01803d191" + # rpcuser : operator + # rpcpassword : otherpassword + + masternodeblskey="masternodeblsprivkey=58af6e39bb4d86b22bda1a02b134c2f5b71caffa1377540b02f7f1ad122f59e0" + + with open(os.path.join(self.options.tmpdir+"/node0", "dash.conf"), 'a', encoding='utf8') as f: + f.write(masternodeblskey+"\n") + f.write(rpcauthplatform+"\n") + f.write(rpcauthoperator+"\n") + + def run_test(self): + url = urllib.parse.urlparse(self.nodes[0].url) + + def send_command(method, params, auth, expected_status, should_not_match=False): + conn = http.client.HTTPConnection(url.hostname, url.port) + conn.connect() + body = {"method": method} + if len(params): + body["params"] = params + conn.request('POST', '/', json.dumps(body), {"Authorization": "Basic " + str_to_b64str(auth)}) + resp = conn.getresponse() + if should_not_match: + assert resp.status != expected_status + else: + assert_equal(resp.status, expected_status) + conn.close() + + rpcuser_authpair_platform = "platform-user:password123" + rpcuser_authpair_operator = "operator:otherpassword" + rpcuser_authpair_wrong = "platform-user:rpcpasswordwrong" + + external_log_str = "HTTP: Calling handler for external user" + always_expected_log_str = "ThreadRPCServer method=" + + self.log.info('Try using a incorrect password for platform-user...') + send_command("getbestblockhash", [], rpcuser_authpair_wrong, 401) + + self.log.info("Check that there's no external queue by default") + with self.nodes[0].assert_debug_log(expected_msgs=[always_expected_log_str], unexpected_msgs = [external_log_str]): + send_command("getbestblockhash", [], rpcuser_authpair_platform, 200) + with self.nodes[0].assert_debug_log(expected_msgs=[always_expected_log_str], unexpected_msgs = [external_log_str]): + send_command("getbestblockhash", [], rpcuser_authpair_operator, 200) + + self.log.info("Restart node with -rpcexternaluser") + self.restart_node(0, extra_args=["-rpcexternaluser=platform-user"]) + + with self.nodes[0].assert_debug_log(expected_msgs=[always_expected_log_str, external_log_str]): + send_command("getbestblockhash", [], rpcuser_authpair_platform, 200) + with self.nodes[0].assert_debug_log(expected_msgs=[always_expected_log_str], unexpected_msgs = [external_log_str]): + send_command("getbestblockhash", [], rpcuser_authpair_operator, 200) + + self.log.info("Restart node with multiple external users") + self.restart_node(0, extra_args=["-rpcexternaluser=platform-user,operator"]) + with self.nodes[0].assert_debug_log(expected_msgs=[always_expected_log_str, external_log_str]): + send_command("getbestblockhash", [], rpcuser_authpair_platform, 200) + with self.nodes[0].assert_debug_log(expected_msgs=[always_expected_log_str, external_log_str]): + send_command("getbestblockhash", [], rpcuser_authpair_operator, 200) + + +if __name__ == '__main__': + HTTPBasicsTest().main() diff --git a/test/functional/test_runner.py b/test/functional/test_runner.py index 3af377ba30dde..992d40072c54c 100755 --- a/test/functional/test_runner.py +++ b/test/functional/test_runner.py @@ -310,7 +310,8 @@ 'wallet_send.py --descriptors', 'wallet_create_tx.py --descriptors', 'p2p_fingerprint.py', - 'rpc_platform_filter.py', + 'rpc_deprecated_platform_filter.py', + 'rpc_external_queue.py', 'rpc_wipewallettxes.py', 'feature_dip0020_activation.py', 'feature_uacomment.py',