15443674054

Committed 04 Jun 2025 01:30PM UTC coverage: 72.396% (+0.07%) from 72.329%

Build # 15443674054

Build Type

push

github

Committed by

web-flow

Commit Message

fix: remove code for deleting stale peers (#7184)

This code is no longer needed since the switch to sqlite

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **Refactor**
- Removed the automatic deletion of stale peers from the connectivity
and peer management processes. The system will no longer periodically
delete inactive or stale peers from the database.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Run Details

2 of 2 new or added lines in 1 file covered. (100.0%)

625 existing lines in 43 files now uncovered.

81294 of 112290 relevant lines covered (72.4%)

243381.89 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

47.92

/base_layer/core/src/base_node/sync/hooks.rs

//  Copyright 2020, The Tari Project
//
//  Redistribution and use in source and binary forms, with or without modification, are permitted provided that the
//  following conditions are met:
//
//  1. Redistributions of source code must retain the above copyright notice, this list of conditions and the following
//  disclaimer.
//
//  2. Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the
//  following disclaimer in the documentation and/or other materials provided with the distribution.
//
//  3. Neither the name of the copyright holder nor the names of its contributors may be used to endorse or promote
//  products derived from this software without specific prior written permission.
//
//  THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES,
//  INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
//  DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
//  SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
//  SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
//  WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE
//  USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

#![allow(clippy::type_complexity)]

use std::sync::Arc;

use crate::{
    base_node::sync::{horizon_state_sync::HorizonSyncInfo, SyncPeer},
    blocks::ChainBlock,
};

#[derive(Default)]
pub(super) struct Hooks {
    on_starting: Vec<Box<dyn FnOnce(&SyncPeer) + Send + Sync>>,
    on_progress_header: Vec<Box<dyn Fn(u64, u64, &SyncPeer) + Send + Sync>>,
    on_progress_block: Vec<Box<dyn Fn(Arc<ChainBlock>, u64, &SyncPeer) + Send + Sync>>,
    on_progress_horizon_sync: Vec<Box<dyn Fn(HorizonSyncInfo) + Send + Sync>>,
    on_complete: Vec<Box<dyn Fn(Arc<ChainBlock>, u64) + Send + Sync>>,
    on_rewind: Vec<Box<dyn Fn(Vec<Arc<ChainBlock>>) + Send + Sync>>,
}

impl Hooks {
    pub fn add_on_starting_hook<H>(&mut self, hook: H)
    where H: FnOnce(&SyncPeer) + Send + Sync + 'static {
        self.on_starting.push(Box::new(hook));
    }

    pub fn call_on_starting_hook(&mut self, sync_peer: &SyncPeer) {
        self.on_starting.drain(..).for_each(|f| (f)(sync_peer));
    }

    pub fn add_on_progress_header_hook<H>(&mut self, hook: H)
    where H: Fn(u64, u64, &SyncPeer) + Send + Sync + 'static {
        self.on_progress_header.push(Box::new(hook));
    }

    pub fn call_on_progress_header_hooks(&self, local_height: u64, remote_height: u64, sync_peer: &SyncPeer) {
        self.on_progress_header
            .iter()
            .for_each(|f| (*f)(local_height, remote_height, sync_peer));
    }

    pub fn add_on_progress_block_hook<H>(&mut self, hook: H)
    where H: Fn(Arc<ChainBlock>, u64, &SyncPeer) + Send + Sync + 'static {
        self.on_progress_block.push(Box::new(hook));
    }

    pub fn call_on_progress_block_hooks(&self, block: Arc<ChainBlock>, remote_tip_height: u64, sync_peer: &SyncPeer) {
        self.on_progress_block
            .iter()
            .for_each(|f| (*f)(block.clone(), remote_tip_height, sync_peer));
    }

    pub fn add_on_progress_horizon_hook<H>(&mut self, hook: H)
    where H: Fn(HorizonSyncInfo) + Send + Sync + 'static {
        self.on_progress_horizon_sync.push(Box::new(hook));
    }

    pub fn call_on_progress_horizon_hooks(&self, info: HorizonSyncInfo) {
        self.on_progress_horizon_sync.iter().for_each(|f| (*f)(info.clone()));
    }

    pub fn add_on_complete_hook<H>(&mut self, hook: H)
    where H: Fn(Arc<ChainBlock>, u64) + Send + Sync + 'static {
        self.on_complete.push(Box::new(hook));
    }

    pub fn call_on_complete_hooks(&self, final_block: Arc<ChainBlock>, starting_height: u64) {
        self.on_complete
            .iter()
            .for_each(|f| (*f)(final_block.clone(), starting_height));
    }

    pub fn add_on_rewind_hook<H>(&mut self, hook: H)
    where H: Fn(Vec<Arc<ChainBlock>>) + Send + Sync + 'static {
        self.on_rewind.push(Box::new(hook));
    }

    pub fn call_on_rewind_hooks(&mut self, blocks: Vec<Arc<ChainBlock>>) {
        self.on_rewind.iter().for_each(|f| (*f)(blocks.clone()));
    }
}

1	// Copyright 2020, The Tari Project
2	//
3	// Redistribution and use in source and binary forms, with or without modification, are permitted provided that the
4	// following conditions are met:
5	//
6	// 1. Redistributions of source code must retain the above copyright notice, this list of conditions and the following
7	// disclaimer.
8	//
9	// 2. Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the
10	// following disclaimer in the documentation and/or other materials provided with the distribution.
11	//
12	// 3. Neither the name of the copyright holder nor the names of its contributors may be used to endorse or promote
13	// products derived from this software without specific prior written permission.
14	//
15	// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES,
16	// INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
17	// DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
18	// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
19	// SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
20	// WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE
21	// USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
22
23	#![allow(clippy::type_complexity)]
24
25	use std::sync::Arc;
26
27	use crate::{
28	base_node::sync::{horizon_state_sync::HorizonSyncInfo, SyncPeer},
29	blocks::ChainBlock,
30	};
31
32	#[derive(Default)]
33	pub(super) struct Hooks {
34	on_starting: Vec<Box<dyn FnOnce(&SyncPeer) + Send + Sync>>,
35	on_progress_header: Vec<Box<dyn Fn(u64, u64, &SyncPeer) + Send + Sync>>,
36	on_progress_block: Vec<Box<dyn Fn(Arc<ChainBlock>, u64, &SyncPeer) + Send + Sync>>,
37	on_progress_horizon_sync: Vec<Box<dyn Fn(HorizonSyncInfo) + Send + Sync>>,
38	on_complete: Vec<Box<dyn Fn(Arc<ChainBlock>, u64) + Send + Sync>>,
39	on_rewind: Vec<Box<dyn Fn(Vec<Arc<ChainBlock>>) + Send + Sync>>,
40	}
41
42	impl Hooks {
43	pub fn add_on_starting_hook<H>(&mut self, hook: H)	12✔
44	where H: FnOnce(&SyncPeer) + Send + Sync + 'static {	12✔
45	self.on_starting.push(Box::new(hook));	12✔
46	}	12✔
47
48	pub fn call_on_starting_hook(&mut self, sync_peer: &SyncPeer) {	12✔
49	self.on_starting.drain(..).for_each(\|f\| (f)(sync_peer));	12✔
50	}	12✔
51
52	pub fn add_on_progress_header_hook<H>(&mut self, hook: H)	12✔
53	where H: Fn(u64, u64, &SyncPeer) + Send + Sync + 'static {	12✔
54	self.on_progress_header.push(Box::new(hook));	12✔
55	}	12✔
56
57	pub fn call_on_progress_header_hooks(&self, local_height: u64, remote_height: u64, sync_peer: &SyncPeer) {	2✔
58	self.on_progress_header	2✔
59	.iter()	2✔
60	.for_each(\|f\| (*f)(local_height, remote_height, sync_peer));	2✔
61	}	2✔
62
UNCOV 63	pub fn add_on_progress_block_hook<H>(&mut self, hook: H)	×
UNCOV 64	where H: Fn(Arc<ChainBlock>, u64, &SyncPeer) + Send + Sync + 'static {	×
UNCOV 65	self.on_progress_block.push(Box::new(hook));	×
UNCOV 66	}	×
67
UNCOV 68	pub fn call_on_progress_block_hooks(&self, block: Arc<ChainBlock>, remote_tip_height: u64, sync_peer: &SyncPeer) {	×
UNCOV 69	self.on_progress_block	×
UNCOV 70	.iter()	×
UNCOV 71	.for_each(\|f\| (*f)(block.clone(), remote_tip_height, sync_peer));	×
UNCOV 72	}	×
73
74	pub fn add_on_progress_horizon_hook<H>(&mut self, hook: H)	×
75	where H: Fn(HorizonSyncInfo) + Send + Sync + 'static {	×
76	self.on_progress_horizon_sync.push(Box::new(hook));	×
77	}	×
78
79	pub fn call_on_progress_horizon_hooks(&self, info: HorizonSyncInfo) {	×
80	self.on_progress_horizon_sync.iter().for_each(\|f\| (*f)(info.clone()));	×
81	}	×
82
UNCOV 83	pub fn add_on_complete_hook<H>(&mut self, hook: H)	×
UNCOV 84	where H: Fn(Arc<ChainBlock>, u64) + Send + Sync + 'static {	×
UNCOV 85	self.on_complete.push(Box::new(hook));	×
UNCOV 86	}	×
87
UNCOV 88	pub fn call_on_complete_hooks(&self, final_block: Arc<ChainBlock>, starting_height: u64) {	×
UNCOV 89	self.on_complete	×
UNCOV 90	.iter()	×
UNCOV 91	.for_each(\|f\| (*f)(final_block.clone(), starting_height));	×
UNCOV 92	}	×
93
94	pub fn add_on_rewind_hook<H>(&mut self, hook: H)	12✔
95	where H: Fn(Vec<Arc<ChainBlock>>) + Send + Sync + 'static {	12✔
96	self.on_rewind.push(Box::new(hook));	12✔
97	}	12✔
98
99	pub fn call_on_rewind_hooks(&mut self, blocks: Vec<Arc<ChainBlock>>) {	1✔
100	self.on_rewind.iter().for_each(\|f\| (*f)(blocks.clone()));	1✔
101	}	1✔
102	}

tari-project / tari / 15443674054

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous