Skip to content

Commit 33f34c7

Browse files
committed
daemon: log when we cannot parse credential request for any reason
1 parent ec338e9 commit 33f34c7

1 file changed

Lines changed: 7 additions & 10 deletions

File tree

credentialsd/src/dbus/gateway.rs

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -257,16 +257,13 @@ impl<C: CredentialRequestController + Send + Sync + 'static> CredentialGateway<C
257257
// - if RP ID is set, but origin's effective domain doesn't match
258258
// - query for related origins, if supported
259259
// - fail if not supported, or if RP ID doesn't match any related origins.
260-
let (make_cred_request, client_data_json) = create_credential_request_try_into_ctap2(
261-
&request,
262-
&request_environment,
263-
)
264-
.map_err(|e| {
265-
if let WebAuthnError::TypeError = e {
266-
tracing::error!("Could not parse passkey creation request. Rejecting request.");
267-
}
268-
e
269-
})?;
260+
let (make_cred_request, client_data_json) =
261+
create_credential_request_try_into_ctap2(&request, &request_environment)
262+
.inspect_err(|_| {
263+
tracing::error!(
264+
"Could not parse passkey creation request. Rejecting request."
265+
);
266+
})?;
270267
if make_cred_request.algorithms.is_empty() {
271268
tracing::info!("No supported algorithms given in request. Rejecting request.");
272269
return Err(Error::NotSupportedError);

0 commit comments

Comments
 (0)