Explicit null check

dodobrands · Jun 4, 2024 · cca6f86 · cca6f86
1 parent ddde885
commit cca6f86
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/demo/WebAuthn.Net.Demo.Mvc/wwwroot/js/lib.js b/demo/WebAuthn.Net.Demo.Mvc/wwwroot/js/lib.js
@@ -108,7 +108,8 @@ const Alerts = {
     usernameInputEmpty: () => alert("Username input is empty"),
     credentialsGetApiNull: () => alert("navigator.credentials.get returned null"),
     credentialsCreateApiNull: () => alert("navigator.credentials.create returned null"),
-    getAuthenticatorDataInvalid: () => alert("Invalid data from getAuthenticatorData() method. Expected ArrayBuffer")
+    getAuthenticatorDataInvalid: () => alert("Invalid data from getAuthenticatorData() method. Expected ArrayBuffer"),
+    getPublicKeyInvalid: () => alert("Invalid data from getPublicKey() method. Expected ArrayBuffer")
 };
 
 // API
@@ -143,10 +144,13 @@ const API = {
             if (newCredential.response.getPublicKey) {
                 const responsePublicKey = newCredential.response.getPublicKey();
                 const isValid = responsePublicKey instanceof ArrayBuffer;
-                if (isValid) {
+                if (responsePublicKey === null) {
+                    publicKey = null;
+                } else if (isValid) {
                     publicKey = coerceToBase64Url(responsePublicKey);
                 } else {
-                    publicKey = null;
+                    Alerts.getPublicKeyInvalid();
+                    return;
                 }
             }