server: Keep a ref to the connection in ServiceManager

This removes the connection attribute from Service.
And add object_server()

Signed-off-by: Dhanuka Warusadura <dhanuka@gnome.org>

server/src/service.rs

@@ -27,8 +27,6 @@ pub struct Service {
     collections: Arc<Mutex<Vec<OwnedObjectPath>>>,
     // Other attributes
     manager: Arc<Mutex<ServiceManager>>,
-    #[allow(unused)]
-    connection: zbus::Connection,
 }
 
 #[zbus::interface(name = "org.freedesktop.Secret.Service")]
@@ -206,8 +204,7 @@ impl Service {
         let object_server = connection.object_server();
         let service = Self {
             collections: Default::default(),
-            manager: Default::default(),
-            connection: connection.clone(),
+            manager: Arc::new(Mutex::new(ServiceManager::new(connection.clone()))),
         };
 
         object_server

server/src/service_manager.rs

@@ -2,17 +2,29 @@
 
 use std::{collections::HashMap, sync::Arc};
 
-use zbus::zvariant::OwnedObjectPath;
+use zbus::{zvariant::OwnedObjectPath, Connection};
 
 use crate::session::Session;
 
-#[derive(Debug, Default)]
+#[derive(Debug)]
 pub struct ServiceManager {
+    connection: Connection,
     // sessions mapped to their corresponding object path on the bus
     sessions: HashMap<OwnedObjectPath, Arc<Session>>,
 }
 
 impl ServiceManager {
+    pub fn new(connection: Connection) -> Self {
+        Self {
+            sessions: Default::default(),
+            connection,
+        }
+    }
+
+    pub fn object_server(&self) -> &zbus::ObjectServer {
+        self.connection.object_server()
+    }
+
     pub fn n_sessions(&self) -> usize {
         self.sessions.len()
     }