emailengine-app 2.61.3 → 2.61.4

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [2.61.4](https://github.com/postalsys/emailengine/compare/v2.61.3...v2.61.4) (2026-01-14)
+
+
+### Bug Fixes
+
+* handle delegation errors in loadAccountData, isApiClient, and UI listing ([c306618](https://github.com/postalsys/emailengine/commit/c306618a37428bbf088ed1289bf832e778f14b9c))
+* show Failed status for accounts with IMAP disabled due to auth errors ([e6b687d](https://github.com/postalsys/emailengine/commit/e6b687d458cfd080795f7176f862109e2c299fff))
+
 ## [2.61.3](https://github.com/postalsys/emailengine/compare/v2.61.2...v2.61.3) (2026-01-14)
 
 

package/lib/account.js

@@ -697,20 +697,30 @@ class Account {
                 accountData._app = app;
             }
         } else if (accountData.oauth2 && accountData.oauth2.auth && accountData.oauth2.auth.delegatedAccount) {
-            let delegatedAccount = await resolveDelegatedAccount(this.redis, accountData.account);
-            if (delegatedAccount) {
-                accountData.delegatedAccount = delegatedAccount;
-                let delegatedAccountRow = await this.redis.hgetall(`${REDIS_PREFIX}iad:${delegatedAccount}`);
-                let delegatedAccountData = this.unserializeAccountData(delegatedAccountRow);
-                if (delegatedAccountData.oauth2 && delegatedAccountData.oauth2.provider) {
-                    let app = await oauth2Apps.get(delegatedAccountData.oauth2.provider);
-                    if (app) {
-                        accountData._app = app;
-                        if (app.baseScopes === 'api') {
-                            accountData.isApi = true;
+            try {
+                let delegatedAccount = await resolveDelegatedAccount(this.redis, accountData.account);
+                if (delegatedAccount) {
+                    accountData.delegatedAccount = delegatedAccount;
+                    let delegatedAccountRow = await this.redis.hgetall(`${REDIS_PREFIX}iad:${delegatedAccount}`);
+                    let delegatedAccountData = this.unserializeAccountData(delegatedAccountRow);
+                    if (delegatedAccountData.oauth2 && delegatedAccountData.oauth2.provider) {
+                        let app = await oauth2Apps.get(delegatedAccountData.oauth2.provider);
+                        if (app) {
+                            accountData._app = app;
+                            if (app.baseScopes === 'api') {
+                                accountData.isApi = true;
+                            }
                         }
                     }
                 }
+            } catch (err) {
+                this.logger.warn({
+                    msg: 'Failed to resolve delegated account',
+                    account: accountData.account,
+                    delegatedAccount: accountData.oauth2.auth.delegatedAccount,
+                    err
+                });
+                accountData.delegationError = err.message;
             }
         }
 
@@ -2545,17 +2555,22 @@ class Account {
 
     async isApiClient(accountData) {
         if (accountData.oauth2?.auth?.delegatedAccount) {
-            let delegatedAccount = await resolveDelegatedAccount(this.redis, accountData.account);
-            if (delegatedAccount) {
-                accountData.delegatedAccount = delegatedAccount;
-                let delegatedAccountRow = await this.redis.hgetall(`${REDIS_PREFIX}iad:${delegatedAccount}`);
-                let delegatedAccountData = this.unserializeAccountData(delegatedAccountRow);
-                if (delegatedAccountData?.oauth2?.provider) {
-                    let app = await oauth2Apps.get(delegatedAccountData.oauth2.provider);
-                    return app?.baseScopes === 'api';
-                } else {
-                    return false;
+            try {
+                let delegatedAccount = await resolveDelegatedAccount(this.redis, accountData.account);
+                if (delegatedAccount) {
+                    accountData.delegatedAccount = delegatedAccount;
+                    let delegatedAccountRow = await this.redis.hgetall(`${REDIS_PREFIX}iad:${delegatedAccount}`);
+                    let delegatedAccountData = this.unserializeAccountData(delegatedAccountRow);
+                    if (delegatedAccountData?.oauth2?.provider) {
+                        let app = await oauth2Apps.get(delegatedAccountData.oauth2.provider);
+                        return app?.baseScopes === 'api';
+                    } else {
+                        return false;
+                    }
                 }
+            } catch (err) {
+                // Invalid delegation config - treat as non-API client
+                return false;
             }
         }
 

package/lib/routes-ui.js

@@ -618,6 +618,15 @@ function formatAccountData(account, gt) {
             break;
     }
 
+    // Check if IMAP was disabled due to errors - override state label to show error
+    if (account.imap && account.imap.disabled && account.lastErrorState) {
+        account.stateLabel = {
+            type: 'danger',
+            name: 'Failed',
+            error: account.lastErrorState.description || account.lastErrorState.response
+        };
+    }
+
     if (account.oauth2) {
         account.oauth2.scopes = []
             .concat(account.oauth2.scope || [])
@@ -4157,13 +4166,20 @@ ${Buffer.from(data.content, 'base64url').toString('base64')}
 
             for (let account of accounts.accounts) {
                 let accountObject = new Account({ redis, account: account.account });
-                account.data = await accountObject.loadAccountData(null, null, runIndex);
+                try {
+                    account.data = await accountObject.loadAccountData(null, null, runIndex);
 
-                if (account.data && account.data.oauth2 && account.data.oauth2.provider) {
-                    let oauth2App = await oauth2Apps.get(account.data.oauth2.provider);
-                    if (oauth2App) {
-                        account.data.oauth2.app = oauth2App;
+                    if (account.data && account.data.oauth2 && account.data.oauth2.provider) {
+                        let oauth2App = await oauth2Apps.get(account.data.oauth2.provider);
+                        if (oauth2App) {
+                            account.data.oauth2.app = oauth2App;
+                        }
                     }
+                } catch (err) {
+                    // Account has invalid config (e.g., broken delegation)
+                    account.data = {
+                        delegationError: err.message
+                    };
                 }
             }
 

package/lib/ui-routes/account-routes.js

@@ -159,6 +159,15 @@ function formatAccountData(account, gt) {
             break;
     }
 
+    // Check if IMAP was disabled due to errors - override state label to show error
+    if (account.imap && account.imap.disabled && account.lastErrorState) {
+        account.stateLabel = {
+            type: 'danger',
+            name: 'Failed',
+            error: account.lastErrorState.description || account.lastErrorState.response
+        };
+    }
+
     if (account.oauth2) {
         account.oauth2.scopes = []
             .concat(account.oauth2.scope || [])

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "emailengine-app",
-    "version": "2.61.3",
+    "version": "2.61.4",
     "private": false,
     "productTitle": "EmailEngine",
     "description": "Email Sync Engine",