Merge pull request #5582
fcfcc3a
rpc: in/out peers can now return the setting's value (moneromooo-monero)
This commit is contained in:
commit
e241a6280d
|
@ -494,11 +494,14 @@ bool t_command_parser_executor::set_limit_down(const std::vector<std::string>& a
|
|||
|
||||
bool t_command_parser_executor::out_peers(const std::vector<std::string>& args)
|
||||
{
|
||||
if (args.empty()) return false;
|
||||
|
||||
unsigned int limit;
|
||||
bool set = false;
|
||||
uint32_t limit = 0;
|
||||
try {
|
||||
if (!args.empty())
|
||||
{
|
||||
limit = std::stoi(args[0]);
|
||||
set = true;
|
||||
}
|
||||
}
|
||||
|
||||
catch(const std::exception& ex) {
|
||||
|
@ -506,16 +509,19 @@ bool t_command_parser_executor::out_peers(const std::vector<std::string>& args)
|
|||
return false;
|
||||
}
|
||||
|
||||
return m_executor.out_peers(limit);
|
||||
return m_executor.out_peers(set, limit);
|
||||
}
|
||||
|
||||
bool t_command_parser_executor::in_peers(const std::vector<std::string>& args)
|
||||
{
|
||||
if (args.empty()) return false;
|
||||
|
||||
unsigned int limit;
|
||||
bool set = false;
|
||||
uint32_t limit = 0;
|
||||
try {
|
||||
if (!args.empty())
|
||||
{
|
||||
limit = std::stoi(args[0]);
|
||||
set = true;
|
||||
}
|
||||
}
|
||||
|
||||
catch(const std::exception& ex) {
|
||||
|
@ -523,7 +529,7 @@ bool t_command_parser_executor::in_peers(const std::vector<std::string>& args)
|
|||
return false;
|
||||
}
|
||||
|
||||
return m_executor.in_peers(limit);
|
||||
return m_executor.in_peers(set, limit);
|
||||
}
|
||||
|
||||
bool t_command_parser_executor::hard_fork_info(const std::vector<std::string>& args)
|
||||
|
|
|
@ -1466,13 +1466,14 @@ bool t_rpc_command_executor::get_limit_down()
|
|||
return true;
|
||||
}
|
||||
|
||||
bool t_rpc_command_executor::out_peers(uint64_t limit)
|
||||
bool t_rpc_command_executor::out_peers(bool set, uint32_t limit)
|
||||
{
|
||||
cryptonote::COMMAND_RPC_OUT_PEERS::request req;
|
||||
cryptonote::COMMAND_RPC_OUT_PEERS::response res;
|
||||
|
||||
epee::json_rpc::error error_resp;
|
||||
|
||||
req.set = set;
|
||||
req.out_peers = limit;
|
||||
|
||||
std::string fail_message = "Unsuccessful";
|
||||
|
@ -1493,18 +1494,20 @@ bool t_rpc_command_executor::out_peers(uint64_t limit)
|
|||
}
|
||||
}
|
||||
|
||||
tools::msg_writer() << "Max number of out peers set to " << limit << std::endl;
|
||||
const std::string s = res.out_peers == (uint32_t)-1 ? "unlimited" : std::to_string(res.out_peers);
|
||||
tools::msg_writer() << "Max number of out peers set to " << s << std::endl;
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
bool t_rpc_command_executor::in_peers(uint64_t limit)
|
||||
bool t_rpc_command_executor::in_peers(bool set, uint32_t limit)
|
||||
{
|
||||
cryptonote::COMMAND_RPC_IN_PEERS::request req;
|
||||
cryptonote::COMMAND_RPC_IN_PEERS::response res;
|
||||
|
||||
epee::json_rpc::error error_resp;
|
||||
|
||||
req.set = set;
|
||||
req.in_peers = limit;
|
||||
|
||||
std::string fail_message = "Unsuccessful";
|
||||
|
@ -1525,7 +1528,8 @@ bool t_rpc_command_executor::in_peers(uint64_t limit)
|
|||
}
|
||||
}
|
||||
|
||||
tools::msg_writer() << "Max number of in peers set to " << limit << std::endl;
|
||||
const std::string s = res.in_peers == (uint32_t)-1 ? "unlimited" : std::to_string(res.in_peers);
|
||||
tools::msg_writer() << "Max number of in peers set to " << s << std::endl;
|
||||
|
||||
return true;
|
||||
}
|
||||
|
|
|
@ -125,9 +125,9 @@ public:
|
|||
|
||||
bool set_limit(int64_t limit_down, int64_t limit_up);
|
||||
|
||||
bool out_peers(uint64_t limit);
|
||||
bool out_peers(bool set, uint32_t limit);
|
||||
|
||||
bool in_peers(uint64_t limit);
|
||||
bool in_peers(bool set, uint32_t limit);
|
||||
|
||||
bool hard_fork_info(uint8_t version);
|
||||
|
||||
|
|
|
@ -244,7 +244,9 @@ namespace nodetool
|
|||
size_t get_zone_count() const { return m_network_zones.size(); }
|
||||
|
||||
void change_max_out_public_peers(size_t count);
|
||||
uint32_t get_max_out_public_peers() const;
|
||||
void change_max_in_public_peers(size_t count);
|
||||
uint32_t get_max_in_public_peers() const;
|
||||
virtual bool block_host(const epee::net_utils::network_address &adress, time_t seconds = P2P_IP_BLOCKTIME);
|
||||
virtual bool unblock_host(const epee::net_utils::network_address &address);
|
||||
virtual bool block_subnet(const epee::net_utils::ipv4_network_subnet &subnet, time_t seconds = P2P_IP_BLOCKTIME);
|
||||
|
|
|
@ -2336,6 +2336,15 @@ namespace nodetool
|
|||
}
|
||||
}
|
||||
|
||||
template<class t_payload_net_handler>
|
||||
uint32_t node_server<t_payload_net_handler>::get_max_out_public_peers() const
|
||||
{
|
||||
const auto public_zone = m_network_zones.find(epee::net_utils::zone::public_);
|
||||
if (public_zone == m_network_zones.end())
|
||||
return 0;
|
||||
return public_zone->second.m_config.m_net_config.max_out_connection_count;
|
||||
}
|
||||
|
||||
template<class t_payload_net_handler>
|
||||
void node_server<t_payload_net_handler>::change_max_in_public_peers(size_t count)
|
||||
{
|
||||
|
@ -2349,6 +2358,15 @@ namespace nodetool
|
|||
}
|
||||
}
|
||||
|
||||
template<class t_payload_net_handler>
|
||||
uint32_t node_server<t_payload_net_handler>::get_max_in_public_peers() const
|
||||
{
|
||||
const auto public_zone = m_network_zones.find(epee::net_utils::zone::public_);
|
||||
if (public_zone == m_network_zones.end())
|
||||
return 0;
|
||||
return public_zone->second.m_config.m_net_config.max_in_connection_count;
|
||||
}
|
||||
|
||||
template<class t_payload_net_handler>
|
||||
bool node_server<t_payload_net_handler>::set_tos_flag(const boost::program_options::variables_map& vm, int flag)
|
||||
{
|
||||
|
|
|
@ -2095,7 +2095,9 @@ namespace cryptonote
|
|||
bool core_rpc_server::on_out_peers(const COMMAND_RPC_OUT_PEERS::request& req, COMMAND_RPC_OUT_PEERS::response& res, const connection_context *ctx)
|
||||
{
|
||||
PERF_TIMER(on_out_peers);
|
||||
if (req.set)
|
||||
m_p2p.change_max_out_public_peers(req.out_peers);
|
||||
res.out_peers = m_p2p.get_max_out_public_peers();
|
||||
res.status = CORE_RPC_STATUS_OK;
|
||||
return true;
|
||||
}
|
||||
|
@ -2103,7 +2105,9 @@ namespace cryptonote
|
|||
bool core_rpc_server::on_in_peers(const COMMAND_RPC_IN_PEERS::request& req, COMMAND_RPC_IN_PEERS::response& res, const connection_context *ctx)
|
||||
{
|
||||
PERF_TIMER(on_in_peers);
|
||||
if (req.set)
|
||||
m_p2p.change_max_in_public_peers(req.in_peers);
|
||||
res.in_peers = m_p2p.get_max_in_public_peers();
|
||||
res.status = CORE_RPC_STATUS_OK;
|
||||
return true;
|
||||
}
|
||||
|
|
|
@ -1682,8 +1682,10 @@ namespace cryptonote
|
|||
{
|
||||
struct request_t
|
||||
{
|
||||
uint64_t out_peers;
|
||||
bool set;
|
||||
uint32_t out_peers;
|
||||
BEGIN_KV_SERIALIZE_MAP()
|
||||
KV_SERIALIZE_OPT(set, true)
|
||||
KV_SERIALIZE(out_peers)
|
||||
END_KV_SERIALIZE_MAP()
|
||||
};
|
||||
|
@ -1691,9 +1693,11 @@ namespace cryptonote
|
|||
|
||||
struct response_t
|
||||
{
|
||||
uint32_t out_peers;
|
||||
std::string status;
|
||||
|
||||
BEGIN_KV_SERIALIZE_MAP()
|
||||
KV_SERIALIZE(out_peers)
|
||||
KV_SERIALIZE(status)
|
||||
END_KV_SERIALIZE_MAP()
|
||||
};
|
||||
|
@ -1704,8 +1708,10 @@ namespace cryptonote
|
|||
{
|
||||
struct request_t
|
||||
{
|
||||
uint64_t in_peers;
|
||||
bool set;
|
||||
uint32_t in_peers;
|
||||
BEGIN_KV_SERIALIZE_MAP()
|
||||
KV_SERIALIZE_OPT(set, true)
|
||||
KV_SERIALIZE(in_peers)
|
||||
END_KV_SERIALIZE_MAP()
|
||||
};
|
||||
|
@ -1713,9 +1719,11 @@ namespace cryptonote
|
|||
|
||||
struct response_t
|
||||
{
|
||||
uint32_t in_peers;
|
||||
std::string status;
|
||||
|
||||
BEGIN_KV_SERIALIZE_MAP()
|
||||
KV_SERIALIZE(in_peers)
|
||||
KV_SERIALIZE(status)
|
||||
END_KV_SERIALIZE_MAP()
|
||||
};
|
||||
|
|
Loading…
Reference in New Issue