Fix issue with attStmt missing for sites that require it (#73)

Devolutions · Apr 23, 2024 · 14c7ac1 · 14c7ac1
1 parent ac72366
commit 14c7ac1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "slauth"
-version = "0.7.6-beta.0"
+version = "0.7.6-beta.1"
 authors = ["richer <richer.arc@gmail.com>", "LucFauvel <luc.fauvel@hotmail.com>"]
 edition = "2021"
 description = "oath HOTP and TOTP complient implementation"

diff --git a/examples/web-server.rs b/examples/web-server.rs
@@ -43,7 +43,7 @@ impl From<slauth::webauthn::error::Error> for TestError {
 impl Responder for TestError {
     fn respond_with_builder(self, builder: Builder, _ctx: &HttpContext) -> Builder {
         match self {
-            TestError::Slauth(_) => builder.status(500),
+            TestError::Slauth(e) => builder.status(500).body(e.to_string()),
             TestError::Internal => builder.status(500),
         }
     }

diff --git a/src/webauthn/authenticator/mod.rs b/src/webauthn/authenticator/mod.rs
@@ -32,6 +32,7 @@ use crate::webauthn::{
     authenticator::responses::PrivateKeyResponse,
     proto::{
         constants::WEBAUTHN_REQUEST_TYPE_GET,
+        raw_message::AttestationStatement,
         web_message::{get_default_rp_id, AuthenticatorAttestationResponseRaw, PublicKeyCredentialRaw, PublicKeyCredentialRequestOptions},
     },
 };
@@ -213,7 +214,7 @@ impl WebauthnAuthenticator {
             },
             raw_auth_data: vec![],
             fmt: WEBAUTHN_FORMAT_NONE.to_owned(),
-            att_stmt: None,
+            att_stmt: Some(AttestationStatement::None),
         }
         .to_bytes()?;