diff --git a/CHANGELOG.md b/CHANGELOG.md index 8a06a32..66005f8 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,6 +2,7 @@ ## Improvements +- daemon: Don't log error if we get PresenceRequired over hybrid (Thank you, @michaelbeaumont!) - 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!) diff --git a/credentialsd/src/credential_service/hybrid.rs b/credentialsd/src/credential_service/hybrid.rs index c5094e9..97f0c5b 100644 --- a/credentialsd/src/credential_service/hybrid.rs +++ b/credentialsd/src/credential_service/hybrid.rs @@ -4,6 +4,7 @@ use std::fmt::Debug; use async_stream::stream; use futures_lite::Stream; use libwebauthn::{ + UvUpdate, proto::CtapError, transport::{ Channel, ChannelSettings, Device, @@ -347,6 +348,8 @@ async fn handle_hybrid_updates( while let Ok(msg) = ux_update_receiver.recv().await { debug!(?msg, "Received hybrid update"); let new_state: Option = match msg { + // Our UI already indicates this and authenticators over hybrid also generally indicate + CableUxUpdate::UvUpdate(UvUpdate::PresenceRequired) => None, CableUxUpdate::UvUpdate(uv_update) => { error!( "Received unexpected UV update in hybrid handler: {:?}",