From 35b570b05fd6bf80d73968c299a0061cd8fc6c03 Mon Sep 17 00:00:00 2001
From: OFF0 <offbyn@protonmail.com>
Date: Tue, 6 Dec 2022 20:57:20 +0100
Subject: [PATCH] keys: fix key validation

Pub and private key settings only showed errors if both values were
truthy, but it did not validate if one input was falsy, in which
case it only showed the last error.

Removed truthy check as it is not needed.
---
 src/main.js | 22 ++++++++++------------
 1 file changed, 10 insertions(+), 12 deletions(-)

diff --git a/src/main.js b/src/main.js
index e3f0146..d4bedaa 100644
--- a/src/main.js
+++ b/src/main.js
@@ -541,19 +541,17 @@ importBtn.addEventListener('click', () => {
 settingsForm.addEventListener('input', () => validKeys(privateKeyInput.value, pubKeyInput.value));
 
 function validKeys(privatekey, pubkey) {
-  if (pubkey && privatekey) {
-    try {
-      if (getPublicKey(privatekey) === pubkey) {
-        statusMessage.hidden = true;
-        statusMessage.textContent = 'public-key corresponds to private-key';
-        importBtn.removeAttribute('disabled');
-        return true;
-      } else {
-        statusMessage.textContent = 'private-key does not correspond to public-key!'
-      }
-    } catch (e) {
-      statusMessage.textContent = `not a valid private-key: ${e.message || e}`;
+  try {
+    if (getPublicKey(privatekey) === pubkey) {
+      statusMessage.hidden = true;
+      statusMessage.textContent = 'public-key corresponds to private-key';
+      importBtn.removeAttribute('disabled');
+      return true;
+    } else {
+      statusMessage.textContent = 'private-key does not correspond to public-key!'
     }
+  } catch (e) {
+    statusMessage.textContent = `not a valid private-key: ${e.message || e}`;
   }
   statusMessage.hidden = false;
   importBtn.setAttribute('disabled', true);