debug: traefik auth failing in compose

Author: dev-AshishRanjan

src/integrations/traefik.ts

@@ -12,43 +12,60 @@ export class TraefikIntegration extends BaseIntegration {
   async collect(): Promise<TraefikStats | null> {
     if (!this.config.enabled) return null;
 
-    // 1. Build Candidate List (Priority + Fallbacks)
-    // We use a Set to handle duplicates if the user config matches a default
     const candidates = new Set<string>();
 
-    // Priority: User Configured URL
-    if (this.config.url && this.config.url.trim() !== '') {
-      candidates.add(this.config.url);
+    // 1. Prioritize User URL
+    const userUrl = this.config.url && this.config.url.trim() !== '' ? this.config.url : null;
+    if (userUrl) {
+      candidates.add(userUrl);
     }
 
-    // Fallbacks: Auto-Discovery Defaults
-    // Useful for Coolify/Dokploy where 127.0.0.1 might fail inside the container
-    candidates.add('http://127.0.0.1:8080');       // Host Networking
-    candidates.add('http://172.17.0.1:8080');      // Docker Bridge Gateway
-    candidates.add('http://host.docker.internal:8080'); // Docker Desktop
-    candidates.add('http://traefik:8080');         // Internal Service Name
+    // 2. Auto-Discovery Fallbacks
+    // These are tried if the user URL fails (or isn't provided)
+    candidates.add('http://127.0.0.1:8080');
+    candidates.add('http://172.17.0.1:8080');
+    candidates.add('http://host.docker.internal:8080');
+    candidates.add('http://traefik:8080');
 
-    // 2. Try endpoints sequentially
-    let lastError: Error | null = null;
+    let userUrlError: Error | null = null;
+    let anySuccess = false;
 
     for (const url of candidates) {
       try {
         const stats = await this.fetchStats(url);
         if (stats) {
-          return stats; // Success!
+          anySuccess = true;
+          return stats;
         }
       } catch (error: any) {
-        lastError = error;
-        // Continue to next candidate...
+        // If this was the user's specific URL, save the error for reporting
+        if (userUrl && url === userUrl) {
+          userUrlError = error;
+        }
+        // Continue to next candidate
       }
     }
 
-    // 3. Log warning only if ALL candidates failed
-    if (lastError) {
-      const code = (lastError as any).code;
-      // Suppress common connectivity errors to avoid log spam
-      if (code !== 'ECONNREFUSED' && code !== 'ETIMEDOUT' && code !== 'ECONNRESET') {
-        logger.warn(`[Traefik] Failed to collect stats from any candidate. Last error: ${lastError.message}`);
+    // 3. Robust Error Reporting
+    if (!anySuccess) {
+      // If the user explicitly configured a URL, report WHY that one failed.
+      // This hides the noise of "ENOTFOUND traefik" which is expected in fallbacks.
+      if (userUrl && userUrlError) {
+        const code = (userUrlError as any).code;
+        const status = (userUrlError as any).response?.status;
+        const msg = (userUrlError as any).message;
+
+        // Log detailed auth errors
+        if (status === 401 || status === 403) {
+          logger.warn(`[Traefik] Auth failed for configured URL (${userUrl}). Check username/password. (${status})`);
+        } else {
+          logger.warn(`[Traefik] Configured URL (${userUrl}) failed: ${msg} [${code || 'HTTP Error'}]`);
+        }
+      } else {
+        // General failure (Auto-discovery failed everywhere)
+        // We log a generic warning, avoiding specific "ENOTFOUND" noise unless debug
+        // logger.debug(...) could be used here if you had a debug level
+        logger.debug(userUrlError);
       }
     }
 
@@ -72,7 +89,7 @@ export class TraefikIntegration extends BaseIntegration {
     const httpsAgent = new https.Agent({ rejectUnauthorized: false });
 
     const response = await axios.get(endpoint, {
-      timeout: 2000,
+      timeout: 3000,
       ...authConfig,
       httpsAgent
     });