dn724/dncurrency
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions 1

Removing redundant check in telemetry limiter. Time points are default constructed at the lowest time point which will already pass the duration check making it redundant. (#3326)

Browse source
This commit is contained in:
clemahieu 2021-06-15 10:52:29 +02:00 • committed by GitHub
commit c047074a49
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

3
nano/node/bootstrap/bootstrap_server.cpp
View file

@ -246,9 +246,8 @@ void nano::bootstrap_server::receive_header_action (boost::system::error_code co
if (is_realtime_connection ())
{
// Only handle telemetry requests if they are outside of the cutoff time
auto is_very_first_message = last_telemetry_req == std::chrono::steady_clock::time_point{};
auto cache_exceeded = std::chrono::steady_clock::now () >= last_telemetry_req + nano::telemetry_cache_cutoffs::network_to_time (node->network_params.network);
if (is_very_first_message || cache_exceeded)
if (cache_exceeded)
{
last_telemetry_req = std::chrono::steady_clock::now ();
add_request (std::make_unique<nano::telemetry_req> (header));