casdoor 1.2.0 → 1.2.1

package/manifest.json

@@ -1,6 +1,6 @@
 {
-  "name": "Casdoor",
-  "description": "The Casdoor extension can help browser users automatically log into applications protected by Casdoor single-sign-on (SSO) system.",
+  "name": "Casdoor Browser Plugin",
+  "description": "Casdoor Browser Plugin can help the users automatically log into applications protected by Casdoor identity provider.",
   "version": "1.0.0",
   "manifest_version": 3,
   "permissions": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "casdoor",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "The Casdoor extension can help browser users automatically log into applications protected by Casdoor single-sign-on (SSO) system.",
   "scripts": {
     "semantic-release": "semantic-release"

package/src/oauth.js

@@ -29,22 +29,57 @@ chrome.storage.sync.get("endpoint", ({endpoint}) => {
   }
 });
 
-const sdk = new Sdk(CasdoorConfig);
+let sdk;
+
+function getStorageData(keys) {
+  return new Promise((resolve, reject) => {
+    chrome.storage.sync.get(keys, (data) => {
+      if (chrome.runtime.lastError) {
+        reject(chrome.runtime.lastError.message);
+      } else {
+        resolve(data);
+      }
+    });
+  });
+}
+
+function initInputParamSdk(){
+  return new Promise((resolve, reject) => {
+  Promise.all([getStorageData("applicationName"), getStorageData("endpoint")])
+    .then(([applicationData, endpointData]) => {
+      const applicationName = applicationData.applicationName;
+      const endpoint = endpointData.endpoint;
+
+      if (applicationName && endpoint) {
+        CasdoorConfig.applicationName = applicationName;
+        CasdoorConfig.endpoint = endpoint;
+      }
+      sdk = new Sdk(CasdoorConfig);
+      resolve(sdk);
+    })
+    .catch((error) => {
+      console.error("init SDK failed:", error);
+      reject(error);
+    });
+  });
+}
 
 // eslint-disable-next-line no-unused-vars
 function login() {
-  sdk.login((accessToken) => {
-    if (accessToken) {
-      chrome.storage.sync.set({accessToken}, () => {
-        sdk
-          .getUserProfile(accessToken)
-          .then((userProfile) => displayUserProfile(userProfile));
-      });
-      setInputDisabledState(true, "endpoint", "applicationName");
-    } else {
-      alert("Login failed!");
-    }
-  });
+  initInputParamSdk().then((sdk) => {
+    sdk.login((accessToken) => {
+      if (accessToken) {
+        chrome.storage.sync.set({accessToken}, () => {
+          sdk
+            .getUserProfile(accessToken)
+            .then((userProfile) => displayUserProfile(userProfile));
+        });
+        setInputDisabledState(true, "endpoint", "applicationName");
+      } else {
+        alert("Login failed!");
+      }
+    });
+  })
 }
 
 // eslint-disable-next-line no-unused-vars

package/src/popup.js

@@ -47,7 +47,11 @@ document.addEventListener("DOMContentLoaded", function() {
       if (accessToken) {
         logout();
       } else {
-        login();
+        setInput()
+        .then(login())
+        .catch(error => {
+          console.error('Error login:', error);
+      });
       }
     });
   });
@@ -60,9 +64,6 @@ document.addEventListener("DOMContentLoaded", function() {
       applicationNameDom.value = applicationName;
     }
   });
-  applicationNameDom.addEventListener("input", function(e) {
-    chrome.storage.sync.set({applicationName: e.target.value});
-  });
 
   const endpointDom = document.getElementById("endpoint");
   chrome.storage.sync.get("endpoint", ({endpoint}) => {
@@ -70,7 +71,18 @@ document.addEventListener("DOMContentLoaded", function() {
       endpointDom.value = endpoint;
     }
   });
-  endpointDom.addEventListener("input", function(e) {
-    chrome.storage.sync.set({endpoint: e.target.value});
-  });
 });
+
+function setInput() {
+  return new Promise((resolve, reject) => {
+    const endpointDom = document.getElementById("endpoint");
+    const applicationNameDom = document.getElementById("applicationName");
+    chrome.storage.sync.set({ endpoint: endpointDom.value, applicationName: applicationNameDom.value }, function() {
+      if (chrome.runtime.lastError) {
+        reject(chrome.runtime.lastError.message);
+      } else {
+        resolve();
+      }
+    });
+  });
+}

package/src/sdk.js

@@ -125,6 +125,9 @@ class Sdk {
     };
     return fetch(applicationUrl, requestConfig)
       .then((response) => response.json())
-      .catch((error) => console.error(error));
+      .catch((error) => {
+        console.error(error)
+        alert("failed to get application");
+      });
   }
 }