add recovery mechanism & delay client creation

pujitm · pujitm · commit fb72e86dc6b0 · 2025-07-07T11:51:09.000-04:00
diff --git a/packages/unraid-api-plugin-connect/src/connection-status/cloud.service.ts b/packages/unraid-api-plugin-connect/src/connection-status/cloud.service.ts
@@ -235,9 +235,14 @@ export class CloudService {
          * as an error.
          */
         if (local !== network) {
-            this.logger.verbose(
-                `Local and network resolvers returned different IPs for "${hostname}". [local="${local}"] [network="${network}"]`
-            );
+            const error = `Local and network resolvers returned different IPs for "${hostname}". [local="${local ?? 'NOT FOUND'}"] [network="${
+                network ?? 'NOT FOUND'
+            }. Please refresh the page."]`;
+            this.mothership.setConnectionStatus({
+                status: MinigraphStatus.ERROR_RETRYING,
+                error,
+            });
+            throw new Error(error);
         }
 
         return { local, network };
diff --git a/packages/unraid-api-plugin-connect/src/mothership-proxy/graphql.client.ts b/packages/unraid-api-plugin-connect/src/mothership-proxy/graphql.client.ts
@@ -1,4 +1,4 @@
-import { Injectable, Logger, OnModuleDestroy, OnModuleInit } from '@nestjs/common';
+import { Injectable, Logger, OnApplicationBootstrap, OnModuleDestroy, OnModuleInit } from '@nestjs/common';
 import { ConfigService } from '@nestjs/config';
 import { EventEmitter2 } from '@nestjs/event-emitter';
 
@@ -27,7 +27,7 @@ const FIVE_MINUTES_MS = 5 * 60 * 1000;
 type Unsubscribe = () => void;
 
 @Injectable()
-export class MothershipGraphqlClientService implements OnModuleInit, OnModuleDestroy {
+export class MothershipGraphqlClientService implements OnApplicationBootstrap, OnModuleDestroy {
     private logger = new Logger(MothershipGraphqlClientService.name);
     private apolloClient: ApolloClient<NormalizedCacheObject> | null = null;
     private wsClient: Client | null = null;
@@ -56,10 +56,10 @@ export class MothershipGraphqlClientService implements OnModuleInit, OnModuleDes
     /**
      * Initialize the GraphQL client when the module is created
      */
-    async onModuleInit(): Promise<void> {
-        await this.createClientInstance();
+    async onApplicationBootstrap(): Promise<void> {
         this.configService.getOrThrow('API_VERSION');
         this.configService.getOrThrow('MOTHERSHIP_GRAPHQL_LINK');
+        await this.createClientInstance();
     }
 
     /**
