@aikidosec/broker-client 1.0.2 → 1.0.3

package/app/client.js

@@ -83,7 +83,6 @@ async function resolveInternalHostname(hostname) {
       const dnsPromises = await import('dns/promises');
       const result = await dnsPromises.resolve4(hostname);
       const ip = result[0];
-      log.debug(`Resolved ${hostname} to ${ip} (system DNS)`);
       return ip;
     } catch (e) {
       log.error(`Failed to resolve ${hostname}: ${e.message}`);
@@ -162,13 +161,11 @@ function isInternalUrl(url) {
         // Not an IP, allow it through if we have custom DNS configured
         // The actual resolution will happen in the forward_request handler
         if (DNS_SERVERS !== null && DNS_SERVERS.length > 0) {
-          log.debug(`Hostname ${hostname} will be resolved with custom DNS during forward`);
           return true;
         }
         
         // For system DNS, we can't do synchronous lookup here
         // Allow all private hostnames through and validate during forward
-        log.debug(`Hostname ${hostname} will be validated during forward`);
         return true;
       }
     }
@@ -283,6 +280,30 @@ socket.on('disconnect', () => {
   log.warn("Disconnected from broker server");
 });
 
+socket.on('connect_error', (error) => {
+  log.error(`Socket.IO connection error: ${error.message}`);
+  if (error.description) {
+    log.error(`  Description: ${JSON.stringify(error.description)}`);
+  }
+  if (error.context) {
+    log.error(`  Context: ${JSON.stringify(error.context)}`);
+  }
+  log.error(`  Type: ${error.type || 'unknown'}`);
+});
+
+// Manager events (on socket.io)
+socket.io.on('error', (error) => {
+  log.error(`Socket.IO Manager error: ${error.message || JSON.stringify(error)}`);
+});
+
+socket.io.on('reconnect_error', (error) => {
+  log.error(`Socket.IO reconnection error: ${error.message}`);
+});
+
+socket.io.on('reconnect_failed', () => {
+  log.error(`Socket.IO reconnection failed after all attempts`);
+});
+
 socket.on('forward_request', async (data, callback) => {
   /**
    * Receive request from broker server and forward to internal resource
@@ -421,7 +442,14 @@ async function registerWithServer() {
         log.warn(`Registration attempt ${attempt + 1} failed: ${response.status} - ${response.data}`);
       }
     } catch (error) {
-      log.warn(`Registration attempt ${attempt + 1} failed: ${error.name}: ${error.message}`);
+      if (error.response) {
+        // HTTP error response
+        log.warn(`Registration attempt ${attempt + 1} failed: ${error.response.status} - ${JSON.stringify(error.response.data)}`);
+      } else {
+        // Error setting up request
+        log.warn(`Registration attempt ${attempt + 1} failed: ${error.message}`);
+      }
+      
       if (getClientId() !== null) {
         // If we have a client_id already, don't retry
         // we should try once to deal with secret rotation

package/app/resourceManager.js

@@ -102,7 +102,7 @@ export class ResourceManager {
       this._resources = { ...resourcesDict };
       this._saveToFile();
     } else {
-      log.debug("No resource changes detected");
+      log.info("No resource changes detected");
     }
   }
 
@@ -183,7 +183,13 @@ export class ResourceManager {
       }
       
     } catch (error) {
-      log.error(`Error fetching resources from broker: ${error.message}`);
+      if (error.response) {
+        // HTTP error response
+        log.error(`Error fetching resources from broker: ${error.response.status} - ${JSON.stringify(error.response.data)}`);
+      } else {
+        // Error setting up request
+        log.error(`Error fetching resources from broker: ${error.message}`);
+      }
     }
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aikidosec/broker-client",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Aikido Broker Client - Runs in customer network to forward requests to internal resources",
   "main": "app/client.js",
   "type": "module",