Change from eager to lazy evaluation on error branch. Otherwise confu…

…sing error logging appears
Atostek · Mar 7, 2024 · 82ab8b6 · 82ab8b6
1 parent 5768ad6
commit 82ab8b6
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/security/authentication/authentication_builtin/authentication.rs b/src/security/authentication/authentication_builtin/authentication.rs
@@ -175,15 +175,13 @@ impl Authentication for AuthenticationBuiltin {
     //
     let certificate_algorithm = identity_certificate
       .key_algorithm()
-      .ok_or(create_security_error!(
-        "Identity Certificate specifies no public key algorithm"
-      ))
+      .ok_or_else(|| {
+        create_security_error!("Identity Certificate specifies no public key algorithm")
+      })
       .and_then(CertificateAlgorithm::try_from)?;
     let ca_algorithm = identity_ca
       .key_algorithm()
-      .ok_or(create_security_error!(
-        "CA Certificate specifies no public key algorithm"
-      ))
+      .ok_or_else(|| create_security_error!("CA Certificate specifies no public key algorithm"))
       .and_then(CertificateAlgorithm::try_from)?;
 
     let identity_token = BuiltinIdentityToken {