improve cloud resolver error for troubleshooting

Author: pujitm

packages/unraid-api-plugin-connect/src/resolver/cloud.resolver.ts

@@ -27,6 +27,14 @@ export class CloudResolver {
         const minigraphql = this.cloudService.checkMothershipClient();
         const cloud = await this.cloudService.checkCloudConnection();
 
+        const cloudError = cloud.error ? `NETWORK: ${cloud.error}` : '';
+        const miniGraphError = minigraphql.error ? `CLOUD: ${minigraphql.error}` : '';
+
+        let error = cloudError || miniGraphError || undefined;
+        if (cloudError && miniGraphError) {
+            error = `${cloudError}\n${miniGraphError}`;
+        }
+
         return {
             relay: {
                 // Left in for UPC backwards compat.
@@ -38,10 +46,7 @@ export class CloudResolver {
             minigraphql,
             cloud,
             allowedOrigins: this.networkService.getAllowedOrigins(),
-            error:
-                `${
-                    cloud.error ? `NETWORK: ${cloud.error}` : ''
-                }\n${minigraphql.error ? `CLOUD: ${minigraphql.error}` : ''}` || undefined,
+            error,
         };
     }
 }

packages/unraid-api-plugin-connect/src/service/cloud.service.ts

@@ -166,7 +166,11 @@ export class CloudService {
         try {
             ip = await this.checkDns();
         } catch (error) {
-            this.logger.warn('Failed to fetch DNS, but Minigraph is connected - continuing');
+            this.logger.warn(
+                'Failed to fetch DNS, but Minigraph is connected - continuing. Error type: %s',
+                typeof error
+            );
+            this.logger.error(error);
             ip = `ERROR: ${error instanceof Error ? error.message : 'Unknown Error'}`;
             // Clear error since we're actually connected to the cloud.
             // Do not populate the ip cache since we're in a weird state (this is a change from the previous behavior).