fix: make sure to delete tunnels that might not exist anymore

If a worker goes off and on might change tunnel address, and we want to
load balance only on the active tunnels.

Signed-off-by: Ettore Di Giacinto <[email protected]>

core/p2p/federated.go

@@ -54,15 +54,29 @@ func (fs *FederatedServer) RandomServer() string {
 	return tunnelAddresses[rand.IntN(len(tunnelAddresses))]
 }
 
-func (fs *FederatedServer) SelectLeastUsedServer() string {
+func (fs *FederatedServer) syncTableStatus() {
+	fs.Lock()
+	defer fs.Unlock()
+	currentTunnels := make(map[string]struct{})
+
 	for _, v := range GetAvailableNodes(fs.service) {
 		if v.IsOnline() {
 			fs.ensureRecordExist(v.TunnelAddress)
-		} else {
-			delete(fs.requestTable, v.TunnelAddress)
+			currentTunnels[v.TunnelAddress] = struct{}{}
 		}
 	}
 
+	// delete tunnels that don't exist anymore
+	for t := range currentTunnels {
+		if _, ok := fs.requestTable[t]; !ok {
+			delete(fs.requestTable, t)
+		}
+	}
+}
+
+func (fs *FederatedServer) SelectLeastUsedServer() string {
+	fs.syncTableStatus()
+
 	fs.Lock()
 	defer fs.Unlock()