@magentrix-corp/magentrix-cli 1.3.15 → 1.3.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@magentrix-corp/magentrix-cli",
-  "version": "1.3.15",
+  "version": "1.3.16",
   "description": "CLI tool for synchronizing local files with Magentrix cloud platform",
   "main": "index.js",
   "type": "module",

package/utils/cli/checkInstanceUrl.js

@@ -1,3 +1,4 @@
+import { lookup } from 'node:dns/promises';
 import debug from '../debug.js';
 
 /**
@@ -11,11 +12,25 @@ import debug from '../debug.js';
  */
 export const checkInstanceUrl = async (instanceUrl) => {
     debug.log('URL-CHECK', `Checking reachability: GET ${instanceUrl}`);
+
+    // Resolve DNS first to help diagnose hosts file / local server issues
+    const hostname = new URL(instanceUrl).hostname;
+    try {
+        const { address, family } = await lookup(hostname);
+        debug.log('URL-CHECK', `DNS resolved: ${hostname} -> ${address} (IPv${family})`);
+    } catch (dnsErr) {
+        debug.log('URL-CHECK', `DNS lookup failed for ${hostname}: ${dnsErr.code || dnsErr.message}`);
+    }
+
+    const controller = new AbortController();
+    const timeout = setTimeout(() => {
+        debug.log('URL-CHECK', `Timeout after 15s — aborting request to ${instanceUrl}. If this is a local server (hosts file), check: (1) IIS has an HTTPS binding on port 443 for this hostname, (2) the SSL certificate is valid/trusted for "${hostname}", (3) Node.js trusts the certificate (self-signed certs require NODE_EXTRA_CA_CERTS or NODE_TLS_REJECT_UNAUTHORIZED=0)`);
+        controller.abort();
+    }, 15000);
     try {
-        // Native fetch is available in Node 18+; for older versions, use node-fetch.
         const response = await fetch(instanceUrl, {
             method: "GET",
-            // You can add a small timeout here using AbortController if needed.
+            signal: controller.signal,
         });
 
         debug.log('URL-CHECK', `Response: ${response.status} ${response.statusText}`);
@@ -30,7 +45,19 @@ export const checkInstanceUrl = async (instanceUrl) => {
         const cause = err.cause || {};
         const rootCode = cause.code || '';
         const rootMsg = cause.message || '';
-        const detail = rootCode ? `${rootCode}: ${rootMsg}` : (err.message || String(err));
+        const isTimeout = err.name === 'AbortError' || rootCode === 'ABORT_ERR';
+        const isTLS = rootCode === 'UNABLE_TO_VERIFY_LEAF_SIGNATURE' || rootCode === 'CERT_HAS_EXPIRED'
+            || rootCode === 'DEPTH_ZERO_SELF_SIGNED_CERT' || rootCode === 'ERR_TLS_CERT_ALTNAME_INVALID'
+            || rootCode === 'SELF_SIGNED_CERT_IN_CHAIN' || (rootMsg && rootMsg.includes('certificate'));
+
+        let detail;
+        if (isTimeout) {
+            detail = `Connection timed out after 15s. The server at "${hostname}" did not respond. Check the debug log for DNS resolution and troubleshooting hints.`;
+        } else if (isTLS) {
+            detail = `SSL/TLS error (${rootCode}). If using a local server with a self-signed certificate, set NODE_TLS_REJECT_UNAUTHORIZED=0 or use NODE_EXTRA_CA_CERTS. Detail: ${rootMsg}`;
+        } else {
+            detail = rootCode ? `${rootCode}: ${rootMsg}` : (err.message || String(err));
+        }
 
         debug.log('URL-CHECK', `Failed: ${detail}`);
         if (cause.code) debug.log('URL-CHECK', `Error code: ${cause.code}`);
@@ -41,5 +68,7 @@ export const checkInstanceUrl = async (instanceUrl) => {
         throw new Error(
             `Failed to reach instance URL "${instanceUrl}": ${detail}`
         );
+    } finally {
+        clearTimeout(timeout);
     }
 };