address PR comment

(cherry picked from commit 49e858e7d44b6e3ba24a388e2bdbffeb88d9adb8)

1 files changed, 6 insertions, 2 deletions

diff --git a/misc/userscripts/qute-bitwarden b/misc/userscripts/qute-bitwarden
index e94ab0a24..c6407127a 100755
--- a/misc/userscripts/qute-bitwarden
+++ b/misc/userscripts/qute-bitwarden
@@ -153,10 +153,12 @@ def pass_(domain, encoding, auto_lock):
         stderr=subprocess.PIPE,
     )
 
-    if process.returncode:
-        err = process.stderr.decode(encoding).strip()
+    err = process.stderr.decode(encoding).strip()
+    if err:
         msg = 'Bitwarden CLI returned for {:s} - {:s}'.format(domain, err)
         stderr(msg)
+
+    if process.returncode:
         return '[]'
 
     out = process.stdout.decode(encoding).strip()
@@ -177,6 +179,8 @@ def get_totp_code(selection_id, domain_name, encoding, auto_lock):
         # domain_name instead of selection_id to make it more user-friendly
         msg = 'Bitwarden CLI returned for {:s} - {:s}'.format(domain_name, err)
         stderr(msg)
+
+    if process.returncode:
         return '[]'
 
     out = process.stdout.decode(encoding).strip()