From e4714e4b31e2eb88733d63f92c1bf61511d25e83 Mon Sep 17 00:00:00 2001
From: Felix Mosheev <9304194+felixmosh@users.noreply.github.com>
Date: Tue, 12 Apr 2022 09:01:39 +0300
Subject: Change error name to a specific type, fixes #190

---
 .../src/helpers/identifyRegistrationError.ts       | 28 +++++++++++++---------
 1 file changed, 17 insertions(+), 11 deletions(-)

(limited to 'packages/browser/src/helpers/identifyRegistrationError.ts')

diff --git a/packages/browser/src/helpers/identifyRegistrationError.ts b/packages/browser/src/helpers/identifyRegistrationError.ts
index 544953b..5b560e1 100644
--- a/packages/browser/src/helpers/identifyRegistrationError.ts
+++ b/packages/browser/src/helpers/identifyRegistrationError.ts
@@ -20,29 +20,31 @@ export function identifyRegistrationError({
   if (error.name === 'AbortError') {
     if (options.signal === new AbortController().signal) {
       // https://www.w3.org/TR/webauthn-2/#sctn-createCredential (Step 16)
-      return new WebAuthnError('Registration ceremony was sent an abort signal');
+      return new WebAuthnError('Registration ceremony was sent an abort signal', 'AbortError');
     }
   } else if (error.name === 'ConstraintError') {
     if (publicKey.authenticatorSelection?.requireResidentKey === true) {
       // https://www.w3.org/TR/webauthn-2/#sctn-op-make-cred (Step 4)
       return new WebAuthnError(
-        'Discoverable credentials were required but no available authenticator supported it (ConstraintError)',
+        'Discoverable credentials were required but no available authenticator supported it',
+        'ConstraintError'
       );
     } else if (publicKey.authenticatorSelection?.userVerification === 'required') {
       // https://www.w3.org/TR/webauthn-2/#sctn-op-make-cred (Step 5)
       return new WebAuthnError(
-        'User verification was required but no available authenticator supported it (ConstraintError)',
+        'User verification was required but no available authenticator supported it',
+        'ConstraintError'
       );
     }
   } else if (error.name === 'InvalidStateError') {
     // https://www.w3.org/TR/webauthn-2/#sctn-createCredential (Step 20)
     // https://www.w3.org/TR/webauthn-2/#sctn-op-make-cred (Step 3)
-    return new WebAuthnError('The authenticator was previously registered (InvalidStateError)');
+    return new WebAuthnError('The authenticator was previously registered', 'InvalidStateError');
   } else if (error.name === 'NotAllowedError') {
     // https://www.w3.org/TR/webauthn-2/#sctn-createCredential (Step 20)
     // https://www.w3.org/TR/webauthn-2/#sctn-createCredential (Step 21)
     return new WebAuthnError(
-      'User clicked cancel, or the registration ceremony timed out (NotAllowedError)',
+      'User clicked cancel, or the registration ceremony timed out', 'NotAllowedError'
     );
   } else if (error.name === 'NotSupportedError') {
     const validPubKeyCredParams = publicKey.pubKeyCredParams.filter(
@@ -52,35 +54,39 @@ export function identifyRegistrationError({
     if (validPubKeyCredParams.length === 0) {
       // https://www.w3.org/TR/webauthn-2/#sctn-createCredential (Step 10)
       return new WebAuthnError(
-        'No entry in pubKeyCredParams was of type "public-key" (NotSupportedError)',
+        'No entry in pubKeyCredParams was of type "public-key"',
+        'NotSupportedError'
       );
     }
 
     // https://www.w3.org/TR/webauthn-2/#sctn-op-make-cred (Step 2)
     return new WebAuthnError(
-      'No available authenticator supported any of the specified pubKeyCredParams algorithms (NotSupportedError)',
+      'No available authenticator supported any of the specified pubKeyCredParams algorithms',
+      'NotSupportedError'
     );
   } else if (error.name === 'SecurityError') {
     const effectiveDomain = window.location.hostname;
     if (!isValidDomain(effectiveDomain)) {
       // https://www.w3.org/TR/webauthn-2/#sctn-createCredential (Step 7)
-      return new WebAuthnError(`${window.location.hostname} is an invalid domain (SecurityError)`);
+      return new WebAuthnError(`${window.location.hostname} is an invalid domain`, 'SecurityError');
     } else if (publicKey.rp.id !== effectiveDomain) {
       // https://www.w3.org/TR/webauthn-2/#sctn-createCredential (Step 8)
       return new WebAuthnError(
-        `The RP ID "${publicKey.rp.id}" is invalid for this domain (SecurityError)`,
+        `The RP ID "${publicKey.rp.id}" is invalid for this domain`,
+        'SecurityError'
       );
     }
   } else if (error.name === 'TypeError') {
     if (publicKey.user.id.byteLength < 1 || publicKey.user.id.byteLength > 64) {
       // https://www.w3.org/TR/webauthn-2/#sctn-createCredential (Step 5)
-      return new WebAuthnError('User ID was not between 1 and 64 characters (TypeError)');
+      return new WebAuthnError('User ID was not between 1 and 64 characters', 'TypeError');
     }
   } else if (error.name === 'UnknownError') {
     // https://www.w3.org/TR/webauthn-2/#sctn-op-make-cred (Step 1)
     // https://www.w3.org/TR/webauthn-2/#sctn-op-make-cred (Step 8)
     return new WebAuthnError(
-      'The authenticator was unable to process the specified options, or could not create a new credential (UnknownError)',
+      'The authenticator was unable to process the specified options, or could not create a new credential',
+      'UnknownError'
     );
   }
 
-- 
cgit v1.2.3