Do not trigger shar when upload failed due to missing credentials/per…

…missions

Issue #5

Signed-off-by: Jochen <rothjochen@gmail.com>

login.js

@@ -7,15 +7,6 @@ const login = context => {
   return new Promise((resolve, reject) => {
     const credentials = []
 
-    const authWindow = new BrowserWindow({
-      'use-content-size': true,
-      alwaysOnTop: true,
-      autoHideMenuBar: false,
-      webPreferences: {
-        nodeIntegration: false
-      }
-    })
-
     /**
      * Register nc:// protocol
      */
@@ -42,6 +33,15 @@ const login = context => {
       console.error(error)
     })
 
+    const authWindow = new BrowserWindow({
+      'use-content-size': true,
+      alwaysOnTop: true,
+      autoHideMenuBar: false,
+      webPreferences: {
+        nodeIntegration: false
+      }
+    })
+
     const extraHeaders = {
       userAgent: 'KapScreenRecorder',
       extraHeaders: 'OCS-APIREQUEST: true\n ACCEPT_LANGUAGE: "en-US"'

package.json

@@ -1,6 +1,6 @@
 {
   "name": "kap-nextcloud",
-  "version": "1.4.2",
+  "version": "1.4.3",
   "description": "Upload Screencast to your Nextcloud Instance",
   "main": "index.js",
   "scripts": {

upload.js

@@ -25,13 +25,14 @@ const upload = (context, filePath) => {
     if (response.statusCode !== 201) {
       context.config.delete('username')
       context.config.delete('password')
+    } else {
+      context.setProgress('Upload finished', 'completed')
+      share(context, filePath)
     }
 
     if (error) {
       return console.error('Nextcloud upload failed:', error)
     }
-    context.setProgress('Upload finished', 'completed')
-    share(context, filePath)
   })
 }