From 83656ad14bd8f55c34afe2dce054c008bfd5f1d8 Mon Sep 17 00:00:00 2001
From: Erayd <steve@erayd.net>
Date: Thu, 17 Sep 2020 23:10:56 +1200
Subject: [PATCH] Copy OTP to clipboard only after fill (#239)

---
 src/background.js | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/background.js b/src/background.js
index a5be17d..d516544 100644
--- a/src/background.js
+++ b/src/background.js
@@ -812,6 +812,15 @@ async function handleMessage(settings, message, sendResponse) {
 
                 // no need to check filledFields, because fillFields() already throws an error if empty
                 sendResponse({ status: "ok", filledFields: filledFields });
+
+                // copy OTP token after fill
+                if (
+                    settings.enableOTP &&
+                    typeof message.login !== "undefined" &&
+                    message.login.fields.hasOwnProperty("otp")
+                ) {
+                    copyToClipboard(helpers.makeTOTP(message.login.fields.otp.params));
+                }
             } catch (e) {
                 try {
                     sendResponse({
@@ -844,15 +853,6 @@ async function handleMessage(settings, message, sendResponse) {
             });
             break;
     }
-
-    // copy OTP token after fill
-    if (
-        settings.enableOTP &&
-        typeof message.login !== "undefined" &&
-        message.login.fields.hasOwnProperty("otp")
-    ) {
-        copyToClipboard(helpers.makeTOTP(message.login.fields.otp.params));
-    }
 }
 
 /**