Remove base-class

Author: abergs

BlazorWasmDemo/Client/Shared/UserService.cs

@@ -134,11 +134,6 @@ public async Task<string> LoginAsync(string? username)
                 return "No options received";
             }
 
-            if (options.Status != "ok")
-            {
-                return options.ErrorMessage ?? string.Empty;
-            }
-
             // Present options to user and get response (usernameless users will be asked by their authenticator, which credential they want to use to sign the challenge)
             var assertion = await _webAuthn.VerifyAsync(options);
 

BlazorWasmDemo/Server/Controllers/UserController.cs

@@ -126,9 +126,9 @@ public CredentialCreateOptions GetCredentialOptions(
             // 6. return options to client
             return options;
         }
-        catch (Exception e)
+        catch (Exception)
         {
-            return new CredentialCreateOptions { Status = "error", ErrorMessage = FormatException(e) };
+            throw;
         }
     }
 
@@ -152,11 +152,6 @@ public async Task<string> CreateCredentialAsync([FromRoute] string username, [Fr
             // 3. Verify and make the credentials
             var result = await _fido2.MakeNewCredentialAsync(attestationResponse, options, CredentialIdUniqueToUserAsync, cancellationToken: cancellationToken);
 
-            if (result.Status is "error" || result.Result is null)
-            {
-                return result.ErrorMessage ?? string.Empty;
-            }
-
             // 4. Store the credentials in db
             _demoStorage.AddCredentialToUser(options.User, new StoredCredential
             {
@@ -229,9 +224,9 @@ public AssertionOptions MakeAssertionOptions([FromRoute] string? username, [From
             // 5. return options to client
             return options;
         }
-        catch (Exception e)
+        catch (Exception)
         {
-            return new AssertionOptions { Status = "error", ErrorMessage = FormatException(e) };
+            throw;
         }
     }
 
@@ -281,18 +276,12 @@ public async Task<string> MakeAssertionAsync([FromBody] AuthenticatorAssertionRa
                 cancellationToken: cancellationToken);
 
             // 4. Store the updated counter
-            if (res.Status is "ok")
-            {
-                _demoStorage.UpdateCounter(res.CredentialId, res.SignCount);
-                if (res.DevicePublicKey is not null)
-                {
-                    creds.DevicePublicKeys.Add(res.DevicePublicKey);
-                }
-            }
-            else
+            _demoStorage.UpdateCounter(res.CredentialId, res.SignCount);
+            if (res.DevicePublicKey is not null)
             {
-                return $"Error: {res.ErrorMessage}";
+                creds.DevicePublicKeys.Add(res.DevicePublicKey);
             }
+            
 
             // 5. return result to client
             var handler = new JwtSecurityTokenHandler();

Demo/Controller.cs

@@ -81,7 +81,7 @@ public JsonResult MakeCredentialOptions([FromForm] string username,
         }
         catch (Exception e)
         {
-            return Json(new CredentialCreateOptions { Status = "error", ErrorMessage = FormatException(e) });
+            return Json(new  { Status = "error", ErrorMessage = FormatException(e) });
         }
     }
 
@@ -132,7 +132,7 @@ public async Task<JsonResult> MakeCredential([FromBody] AuthenticatorAttestation
         }
         catch (Exception e)
         {
-            return Json(new MakeNewCredentialResult(status: "error", errorMessage: FormatException(e), result: null));
+            return Json(new { status = "error", errorMessage = FormatException(e)});
         }
     }
 
@@ -177,7 +177,7 @@ public ActionResult AssertionOptionsPost([FromForm] string username, [FromForm]
 
         catch (Exception e)
         {
-            return Json(new AssertionOptions { Status = "error", ErrorMessage = FormatException(e) });
+            return Json(new  { Status = "error", ErrorMessage = FormatException(e) });
         }
     }
 
@@ -218,7 +218,7 @@ public async Task<JsonResult> MakeAssertion([FromBody] AuthenticatorAssertionRaw
         }
         catch (Exception e)
         {
-            return Json(new VerifyAssertionResult { Status = "error", ErrorMessage = FormatException(e) });
+            return Json(new { Status = "error", ErrorMessage = FormatException(e) });
         }
     }
 }

Src/Fido2.Models/AssertionOptions.cs

@@ -9,7 +9,7 @@ namespace Fido2NetLib;
 /// <summary>
 /// Sent to the browser when we want to Assert credentials and authenticate a user
 /// </summary>
-public class AssertionOptions : Fido2ResponseBase
+public class AssertionOptions
 {
     /// <summary>
     /// This member represents a challenge that the selected authenticator signs, along with other data, when producing an authentication assertion.
@@ -71,8 +71,6 @@ public static AssertionOptions Create(
     {
         return new AssertionOptions()
         {
-            Status = "ok",
-            ErrorMessage = string.Empty,
             Challenge = challenge,
             Timeout = config.Timeout,
             RpId = config.ServerDomain,

Src/Fido2.Models/CredentialCreateOptions.cs

@@ -6,7 +6,7 @@
 
 namespace Fido2NetLib;
 
-public sealed class CredentialCreateOptions : Fido2ResponseBase
+public sealed class CredentialCreateOptions
 {
     /// <summary>
     /// 
@@ -117,8 +117,6 @@ public static CredentialCreateOptions Create(
     {
         return new CredentialCreateOptions
         {
-            Status = "ok",
-            ErrorMessage = string.Empty,
             Challenge = challenge,
             Rp = new PublicKeyCredentialRpEntity(config.ServerDomain, config.ServerName, config.ServerIcon),
             Timeout = config.Timeout,

Src/Fido2.Models/Fido2ResponseBase.cs

@@ -5,16 +5,9 @@ namespace Fido2NetLib.Objects;
 /// <summary>
 /// Result of parsing and verifying attestation. Used to transport Public Key back to RP.
 /// </summary>
-public sealed class MakeNewCredentialResult : Fido2ResponseBase
+public sealed class MakeNewCredentialResult(RegisteredPublicKeyCredential result)
 {
-    public MakeNewCredentialResult(string status, string errorMessage, RegisteredPublicKeyCredential? result)
-    {
-        Status = status;
-        ErrorMessage = errorMessage;
-        Result = result;
-    }
-
-    public RegisteredPublicKeyCredential? Result { get; }
+    public RegisteredPublicKeyCredential Result { get; } = result;
 
     // todo: add debuginfo?
 }

Src/Fido2.Models/Objects/MakeNewCredentialResult.cs

@@ -5,7 +5,7 @@ namespace Fido2NetLib.Objects;
 /// <summary>
 /// Holds parsed credential data
 /// </summary>
-public class RegisteredPublicKeyCredential : Fido2ResponseBase
+public class RegisteredPublicKeyCredential
 {
     /// <summary>
     /// The type of the public key credential source.

Src/Fido2.Models/Objects/RegisteredPublicKeyCredential.cs

@@ -3,7 +3,7 @@
 /// <summary>
 /// Result of the MakeAssertion verification
 /// </summary>
-public class VerifyAssertionResult : Fido2ResponseBase
+public class VerifyAssertionResult
 {
     public byte[] CredentialId { get; init; }
 

Src/Fido2.Models/Objects/VerifyAssertionResult.cs

@@ -176,7 +176,6 @@ public async Task<VerifyAssertionResult> VerifyAsync(
 
         return new VerifyAssertionResult
         {
-            Status = "ok",
             CredentialId = Raw.Id,
             SignCount = authData.SignCount,
             IsBackedUp = authData.IsBackedUp,