Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

## Improvements

- daemon: Cancel ceremony if UI disappears (Thank you, @michaelbeaumont!)
- daemon: Fix UI service failing to launch when started manually instead of via D-Bus activation.
- daemon: Allow Firefox to contact daemon on more distros with different app IDs (Thank you, @michaelbeaumont!)
- daemon: Fix all subsequent requests being rejected after the UI failed to launch.
Expand Down
36 changes: 34 additions & 2 deletions credentialsd/src/dbus/ui_control.rs
Original file line number Diff line number Diff line change
Expand Up @@ -467,12 +467,22 @@ impl UiController for UiControlServiceClient {
};
subscribe_ui_events(
self.conn.clone(),
sender,
sender.clone(),
session_handle.clone(),
from_ui_tx,
from_ui_tx.clone(),
)
.await?;

// We want to know when this specific peer disconnects
let peer = zbus::proxy::Builder::<zbus::Proxy>::new(&self.conn)
.destination(sender)?
.path(backend_proxy.as_ref().path())?
.interface(backend_proxy.as_ref().interface())?
.build()
.await?;
let owner_changes = peer.receive_owner_changed().await?;
watch_ui_exit(owner_changes, session_handle.clone(), from_ui_tx);

backend_proxy
.create_session(
session_handle.as_ref(),
Expand All @@ -494,6 +504,28 @@ impl UiController for UiControlServiceClient {
}
}

fn watch_ui_exit(
owner_changes: proxy::OwnerChangedStream<'static>,
session_handle: OwnedObjectPath,
tx: mpsc::Sender<UserInteractedEvent>,
) {
tokio::spawn(async move {
let mut owner_changes = Box::pin(owner_changes);
while let Some(owner) = tokio::select! {
// Nothing is listening for ceremony events any more
_ = tx.closed() => None,
owner = owner_changes.next() => owner,
} {
// A well-known name yields `None` once it is released.
if owner.is_none() {
tracing::debug!(session_handle = %session_handle, "UI left the bus, cancelling request");
_ = tx.send(UserInteractedEvent::RequestCancelled).await;
return;
}
}
});
}

async fn subscribe_ui_events(
connection: Connection,
sender: OwnedUniqueName,
Expand Down