From 285249a3841952fdec5f4e491638b76fc4de8534 Mon Sep 17 00:00:00 2001
From: Thomas Eizinger <thomas@eizinger.io>
Date: Thu, 4 Apr 2024 12:09:58 +1100
Subject: [PATCH] fix(relay): only unbind a channel if it is actually bound
 (#4495)

Currently, we are emitting the "Channel is now expired" message multiple
times because we don't filter for the ones we have already unbound.
---
 rust/relay/src/server.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/rust/relay/src/server.rs b/rust/relay/src/server.rs
index 7f8284287..15451a088 100644
--- a/rust/relay/src/server.rs
+++ b/rust/relay/src/server.rs
@@ -460,7 +460,7 @@ where
         for ((client, number), channel) in self
             .channels_by_client_and_number
             .iter_mut()
-            .filter(|(_, c)| c.is_expired(now))
+            .filter(|(_, c)| c.is_expired(now) && c.bound)
         {
             tracing::info!(target: "relay", channel = %number, %client, peer = %channel.peer_address, allocation = %channel.allocation, "Channel is now expired");