use identity.getRedirectUrl() instead of setting it manually

Author: Onatcer

entrypoints/background.ts

@@ -25,14 +25,6 @@ export default defineBackground(() => {
     return [...Array(num)].map(() => Math.random().toString(36)[2]).join("");
   }
 
-  function getRedirectUrl() {
-    const extensionId = browser.runtime.id;
-    if (navigator.userAgent.includes("Firefox")) {
-      return `https://${extensionId}.extensions.allizom.org/`;
-    }
-    return `https://${extensionId}.chromiumapp.org/`;
-  }
-
   // Listen for messages from popup or content scripts
   browser.runtime.onMessage.addListener((message, sender, sendResponse) => {
     if (message.type === "START_OAUTH_FLOW") {
@@ -47,7 +39,7 @@ export default defineBackground(() => {
           const hashed = await sha256(oauthVerifier);
           oauthChallenge = base64urlencode(hashed);
 
-          const redirectUrl = getRedirectUrl();
+          const redirectUrl = browser.identity.getRedirectURL();
           const loginUrl =
             endpoint +
             "/oauth/authorize?client_id=" +

entrypoints/utils/oauth.ts

@@ -39,13 +39,8 @@ browser.storage.onChanged.addListener((changes, area) => {
 // Initialize
 loadTokens();
 
-export const getRedirectUrl = () => {
-  const extensionId = browser.runtime.id;
-  if (navigator.userAgent.includes("Firefox")) {
-    return `https://${extensionId}.extensions.allizom.org/`;
-  }
-  return `https://${extensionId}.chromiumapp.org/`;
-};
+// Use browser.identity.getRedirectURL() which works for both Firefox and Chrome
+export const getRedirectUrl = () => browser.identity.getRedirectURL();
 
 export const isLoggedIn = computed(() => !!accessToken.value);
 

package-lock.json

@@ -2,7 +2,7 @@
   "name": "solidtime-browser-extension",
   "description": "Official Browser Extension for solidtime",
   "private": true,
-  "version": "0.0.6",
+  "version": "0.0.7",
   "type": "module",
   "scripts": {
     "dev": "wxt",