change the entra url and changing the electron loopback url when built

actualbudget · Dec 16, 2024 · cf53879 · cf53879
1 parent 45f89b2
commit cf53879
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/packages/desktop-client/src/components/manager/subscribe/OpenIdForm.tsx b/packages/desktop-client/src/components/manager/subscribe/OpenIdForm.tsx
@@ -281,7 +281,7 @@ const openIdProviders: (OpenIdProviderOption | typeof Menu.line)[] = [
     {
       label: 'Microsoft Entra',
       value: 'microsoft',
-      issuer: 'https://login.microsoftonline.com/common/v2.0/',
+      issuer: 'https://login.microsoftonline.com/{tenant-id}',
       clientIdRequired: true,
       clientSecretRequired: true,
       clientSecretDisabled: false,

diff --git a/packages/desktop-electron/index.ts b/packages/desktop-electron/index.ts
@@ -72,7 +72,11 @@ const createOAuthServer = async () => {
 
       const code = query.get('token');
       if (code && clientWin) {
-        clientWin.loadURL(`http://localhost:3001/openid-cb?token=${code}`);
+        if (isDev) {
+          clientWin.loadURL(`http://localhost:3001/openid-cb?token=${code}`);
+        } else {
+          clientWin.loadURL(`app://actual/openid-cb?token=${code}`);
+        }
 
         // Respond to the browser
         res.writeHead(200, { 'Content-Type': 'text/plain' });