fix: update lookup signature to return owned Client instead of reference

Author: itzmanish

moq-relay-ietf/src/bin/moq-relay-ietf/file_coordinator.rs

@@ -179,15 +179,15 @@ impl Coordinator for FileCoordinator {
         Ok(())
     }
 
-    async fn lookup<'a>(
+    async fn lookup(
         &self,
         namespace: &TrackNamespace,
-    ) -> CoordinatorResult<(NamespaceOrigin, Option<&'a Client>)> {
+    ) -> CoordinatorResult<(NamespaceOrigin, Option<Client>)> {
         let namespace = namespace.clone();
         let file_path = self.file_path.clone();
 
         let result = tokio::task::spawn_blocking(
-            move || -> Result<Option<(NamespaceOrigin, Option<&Client>)>> {
+            move || -> Result<Option<(NamespaceOrigin, Option<Client>)>> {
                 let file = OpenOptions::new()
                     .read(true)
                     .write(true)

moq-relay-ietf/src/coordinator.rs

@@ -171,10 +171,10 @@ pub trait Coordinator: Send + Sync {
     ///
     /// - `Ok(NamespaceOrigin, Option<quic::Client>)` - Namespace origin and optional client if available
     /// - `Err` - Namespace not found anywhere
-    async fn lookup<'a>(
+    async fn lookup(
         &self,
         namespace: &TrackNamespace,
-    ) -> CoordinatorResult<(NamespaceOrigin, Option<&'a quic::Client>)>;
+    ) -> CoordinatorResult<(NamespaceOrigin, Option<quic::Client>)>;
 
     /// Graceful shutdown of the coordinator.
     ///

moq-relay-ietf/src/remote.rs

@@ -56,7 +56,7 @@ impl RemoteManager {
         let url = origin.url();
 
         // Get or create a connection to the remote relay
-        let remote = self.get_or_connect(&url, client).await?;
+        let remote = self.get_or_connect(&url, client.as_ref()).await?;
 
         // Subscribe to the track on the remote
         remote.subscribe(namespace, track_name).await