IntenseWebs/veilid
3
0
Fork 0
mirror of https://gitlab.com/veilid/veilid.git synced 2024-11-24 09:50:16 -06:00
Code Issues Packages Projects Releases Wiki Activity

consider nodes with allocated relays as disqualified from being a relay themselves

Browse Source
This commit is contained in:
Christien Rioux 2024-05-05 16:52:35 -04:00
parent 55de1dcb5c
commit aec31443a0
1 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
veilid-core/src/routing_table/tasks/relay_management.rs
View File

@ -163,10 +163,21 @@ impl RoutingTable {
if e.has_node_info(RoutingDomain::LocalNetwork.into()) {
return false;
}
let Some(node_info) = e.node_info(RoutingDomain::PublicInternet) else {
let Some(signed_node_info) = e.signed_node_info(RoutingDomain::PublicInternet) else {
return false;
};
// Until we have a way of reducing a SignedRelayedNodeInfo to a SignedDirectNodeInfo
// See https://gitlab.com/veilid/veilid/-/issues/381
// We should consider nodes with allocated relays as disqualified from being a relay themselves
// due to limitations in representing the PeerInfo for relays that also have relays.
let node_info = match signed_node_info {
SignedNodeInfo::Direct(d) => d.node_info(),
SignedNodeInfo::Relayed(_) => {
return false;
}
};
// Disqualify nodes that don't have relay capability or require a relay themselves
if !(node_info.has_capability(CAP_RELAY) && node_info.is_fully_direct_inbound()) {
// Needs to be able to accept packets to relay directly