backend: Pass handle token to method calls for easier identification and association with their backend logic

backend-demo/src/account.rs

@@ -4,7 +4,7 @@ use ashpd::{
         request::RequestImpl,
         Result,
     },
-    desktop::account::UserInformation,
+    desktop::{account::UserInformation, HandleToken},
     AppID, WindowIdentifierType,
 };
 use async_trait::async_trait;
@@ -14,8 +14,8 @@ pub struct Account;
 
 #[async_trait]
 impl RequestImpl for Account {
-    async fn close(&self) {
-        tracing::debug!("IN Close()");
+    async fn close(&self, token: HandleToken) {
+        tracing::debug!("IN Close(): {token}");
     }
 }
 
@@ -39,6 +39,7 @@ mod fdo_account {
 impl AccountImpl for Account {
     async fn get_user_information(
         &self,
+        _token: HandleToken,
         _app_id: Option<AppID>,
         _window_identifier: Option<WindowIdentifierType>,
         _options: UserInformationOptions,

backend-demo/src/screenshot.rs

@@ -4,7 +4,7 @@ use ashpd::{
         screenshot::{ColorOptions, ScreenshotImpl, ScreenshotOptions},
         Result,
     },
-    desktop::{screenshot::Screenshot as ScreenshotResponse, Color},
+    desktop::{screenshot::Screenshot as ScreenshotResponse, Color, HandleToken},
     AppID, WindowIdentifierType,
 };
 use async_trait::async_trait;
@@ -14,15 +14,16 @@ pub struct Screenshot;
 
 #[async_trait]
 impl RequestImpl for Screenshot {
-    async fn close(&self) {
-        tracing::debug!("IN Close()");
+    async fn close(&self, token: HandleToken) {
+        tracing::debug!("IN Close(): {token}");
     }
 }
 
 #[async_trait]
 impl ScreenshotImpl for Screenshot {
     async fn screenshot(
         &self,
+        _token: HandleToken,
         _app_id: Option<AppID>,
         _window_identifier: Option<WindowIdentifierType>,
         _options: ScreenshotOptions,
@@ -34,6 +35,7 @@ impl ScreenshotImpl for Screenshot {
 
     async fn pick_color(
         &self,
+        _token: HandleToken,
         _app_id: Option<AppID>,
         _window_identifier: Option<WindowIdentifierType>,
         _options: ColorOptions,

backend-demo/src/secret.rs

@@ -2,6 +2,7 @@ use std::collections::HashMap;
 
 use ashpd::{
     backend::{request::RequestImpl, secret::SecretImpl, Result},
+    desktop::HandleToken,
     zbus::zvariant::OwnedValue,
     AppID,
 };
@@ -12,15 +13,16 @@ pub struct Secret;
 
 #[async_trait]
 impl RequestImpl for Secret {
-    async fn close(&self) {
-        tracing::debug!("IN Close()");
+    async fn close(&self, token: HandleToken) {
+        tracing::debug!("IN Close(): {token}");
     }
 }
 
 #[async_trait]
 impl SecretImpl for Secret {
     async fn retrieve(
         &self,
+        _token: HandleToken,
         _app_id: AppID,
         _fd: std::os::fd::OwnedFd,
     ) -> Result<HashMap<String, OwnedValue>> {

backend-demo/src/settings.rs

@@ -5,7 +5,10 @@ use ashpd::{
         request::RequestImpl,
         settings::{SettingsImpl, SettingsSignalEmitter},
     },
-    desktop::settings::{ColorScheme, Namespace, APPEARANCE_NAMESPACE, COLOR_SCHEME_KEY},
+    desktop::{
+        settings::{ColorScheme, Namespace, APPEARANCE_NAMESPACE, COLOR_SCHEME_KEY},
+        HandleToken,
+    },
     zbus::zvariant::OwnedValue,
     PortalError,
 };
@@ -20,8 +23,8 @@ pub struct Settings {
 
 #[async_trait]
 impl RequestImpl for Settings {
-    async fn close(&self) {
-        tracing::debug!("IN Close()");
+    async fn close(&self, token: HandleToken) {
+        tracing::debug!("IN Close(): {token}");
     }
 }
 

backend-demo/src/wallpaper.rs

@@ -4,6 +4,7 @@ use ashpd::{
         wallpaper::{WallpaperImpl, WallpaperOptions},
         Result,
     },
+    desktop::HandleToken,
     AppID, WindowIdentifierType,
 };
 use async_trait::async_trait;
@@ -13,15 +14,16 @@ pub struct Wallpaper;
 
 #[async_trait]
 impl RequestImpl for Wallpaper {
-    async fn close(&self) {
-        tracing::debug!("IN Close()");
+    async fn close(&self, token: HandleToken) {
+        tracing::debug!("IN Close(): {token}");
     }
 }
 
 #[async_trait]
 impl WallpaperImpl for Wallpaper {
     async fn with_uri(
         &self,
+        _token: HandleToken,
         _app_id: Option<AppID>,
         _window_identifier: Option<WindowIdentifierType>,
         _uri: url::Url,

src/backend/access.rs

@@ -7,7 +7,7 @@ use crate::{
         request::{Request, RequestImpl},
         MaybeAppID, MaybeWindowIdentifier, Result,
     },
-    desktop::{file_chooser::Choice, request::Response, Icon},
+    desktop::{file_chooser::Choice, request::Response, HandleToken, Icon},
     zvariant::{self, DeserializeDict, OwnedObjectPath, SerializeDict},
     AppID, WindowIdentifierType,
 };
@@ -63,8 +63,10 @@ impl AccessResponse {
 
 #[async_trait]
 pub trait AccessImpl: RequestImpl {
+    #[allow(clippy::too_many_arguments)]
     async fn access_dialog(
         &self,
+        token: HandleToken,
         app_id: Option<AppID>,
         window_identifier: Option<WindowIdentifierType>,
         title: String,
@@ -109,10 +111,11 @@ impl AccessInterface {
         Request::spawn(
             "Access::AccessDialog",
             &self.cnx,
-            handle,
+            handle.clone(),
             Arc::clone(&self.imp),
             async move {
                 imp.access_dialog(
+                    HandleToken::try_from(&handle).unwrap(),
                     app_id.inner(),
                     window_identifier.inner(),
                     title,

src/backend/account.rs

@@ -7,7 +7,7 @@ use crate::{
         request::{Request, RequestImpl},
         MaybeAppID, MaybeWindowIdentifier, Result,
     },
-    desktop::{account::UserInformation, request::Response},
+    desktop::{account::UserInformation, request::Response, HandleToken},
     zvariant::{DeserializeDict, OwnedObjectPath, Type},
     AppID, WindowIdentifierType,
 };
@@ -28,6 +28,7 @@ impl UserInformationOptions {
 pub trait AccountImpl: RequestImpl {
     async fn get_user_information(
         &self,
+        token: HandleToken,
         app_id: Option<AppID>,
         window_identifier: Option<WindowIdentifierType>,
         options: UserInformationOptions,
@@ -66,11 +67,16 @@ impl AccountInterface {
         Request::spawn(
             "Account::GetUserInformation",
             &self.cnx,
-            handle,
+            handle.clone(),
             Arc::clone(&self.imp),
             async move {
-                imp.get_user_information(app_id.inner(), window_identifier.inner(), options)
-                    .await
+                imp.get_user_information(
+                    HandleToken::try_from(&handle).unwrap(),
+                    app_id.inner(),
+                    window_identifier.inner(),
+                    options,
+                )
+                .await
             },
         )
         .await

src/backend/app_chooser.rs

@@ -7,7 +7,7 @@ use crate::{
         request::{Request, RequestImpl},
         MaybeAppID, MaybeWindowIdentifier,
     },
-    desktop::Response,
+    desktop::{HandleToken, Response},
     zbus::object_server::{InterfaceRef, ObjectServer},
     zvariant::{DeserializeDict, OwnedObjectPath, SerializeDict, Type},
     ActivationToken, AppID, PortalError, WindowIdentifierType,
@@ -79,6 +79,7 @@ impl Choice {
 pub trait AppChooserImpl: RequestImpl {
     async fn choose_application(
         &self,
+        token: HandleToken,
         app_id: Option<AppID>,
         parent_window: Option<WindowIdentifierType>,
         choices: Vec<AppID>,
@@ -124,11 +125,17 @@ impl AppChooserInterface {
         Request::spawn(
             "AppChooser::ChooseApplication",
             &self.cnx,
-            handle,
+            handle.clone(),
             Arc::clone(&self.imp),
             async move {
-                imp.choose_application(app_id.inner(), parent_window.inner(), choices, options)
-                    .await
+                imp.choose_application(
+                    HandleToken::try_from(&handle).unwrap(),
+                    app_id.inner(),
+                    parent_window.inner(),
+                    choices,
+                    options,
+                )
+                .await
             },
         )
         .await

src/backend/background.rs

@@ -6,7 +6,7 @@ use serde_repr::{Deserialize_repr, Serialize_repr};
 
 use crate::{
     backend::request::{Request, RequestImpl},
-    desktop::Response,
+    desktop::{HandleToken, Response},
     zbus::object_server::SignalEmitter,
     zvariant::{OwnedObjectPath, SerializeDict, Type},
     AppID, PortalError,
@@ -56,8 +56,12 @@ pub trait BackgroundSignalEmitter: Send + Sync {
 pub trait BackgroundImpl: RequestImpl {
     async fn get_app_state(&self) -> Result<HashMap<AppID, AppState>, PortalError>;
 
-    async fn notify_background(&self, app_id: AppID, name: &str)
-        -> Result<Background, PortalError>;
+    async fn notify_background(
+        &self,
+        token: HandleToken,
+        app_id: AppID,
+        name: &str,
+    ) -> Result<Background, PortalError>;
 
     async fn enable_autostart(
         &self,
@@ -128,9 +132,12 @@ impl BackgroundInterface {
         Request::spawn(
             "Background::NotifyBackground",
             &self.cnx,
-            handle,
+            handle.clone(),
             Arc::clone(&self.imp),
-            async move { imp.notify_background(app_id, &name).await },
+            async move {
+                imp.notify_background(HandleToken::try_from(&handle).unwrap(), app_id, &name)
+                    .await
+            },
         )
         .await
     }

src/backend/email.rs

@@ -7,7 +7,7 @@ use crate::{
         request::{Request, RequestImpl},
         MaybeAppID, MaybeWindowIdentifier, Result,
     },
-    desktop::request::Response,
+    desktop::{request::Response, HandleToken},
     zvariant::{self, DeserializeDict, OwnedObjectPath},
     ActivationToken, AppID, WindowIdentifierType,
 };
@@ -63,6 +63,7 @@ impl Options {
 pub trait EmailImpl: RequestImpl {
     async fn compose(
         &self,
+        token: HandleToken,
         app_id: Option<AppID>,
         window_identifier: Option<WindowIdentifierType>,
         options: Options,
@@ -100,11 +101,16 @@ impl EmailInterface {
         Request::spawn(
             "Email::ComposeEmail",
             &self.cnx,
-            handle,
+            handle.clone(),
             Arc::clone(&self.imp),
             async move {
-                imp.compose(app_id.inner(), window_identifier.inner(), options)
-                    .await
+                imp.compose(
+                    HandleToken::try_from(&handle).unwrap(),
+                    app_id.inner(),
+                    window_identifier.inner(),
+                    options,
+                )
+                .await
             },
         )
         .await