From 48c031441b6ee91b14f6312d7af372a3e6dfd414 Mon Sep 17 00:00:00 2001 From: brobert Date: Sat, 20 Sep 2025 19:35:23 +0200 Subject: [PATCH] =?UTF-8?q?fix:=20quitar=20autocorrecci=C3=B3n=20de=20URL?= =?UTF-8?q?=20interna=20y=20permitir=20Docker=20en=20producci=C3=B3n?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: aider (openrouter/openai/gpt-5) --- src/services/webhook-manager.ts | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/src/services/webhook-manager.ts b/src/services/webhook-manager.ts index aad54a7..bb3dffd 100644 --- a/src/services/webhook-manager.ts +++ b/src/services/webhook-manager.ts @@ -47,26 +47,17 @@ export class WebhookManager { // Enhanced internal Docker URL validation if (process.env.WEBHOOK_URL.startsWith('http://srv-captain--')) { if (!url.port) { - console.warn('⚠️ Internal Docker URL missing port number - this may cause connection issues'); - // Auto-fix for common CapRover default port - if (process.env.PORT) { - process.env.WEBHOOK_URL = process.env.WEBHOOK_URL.replace( - 'http://srv-captain--', - `http://srv-captain--:${process.env.PORT}/` - ); - console.warn(`⚠️ Auto-corrected URL to: ${process.env.WEBHOOK_URL.substring(0, 30)}...`); - return; // Skip further validation for corrected URL - } + console.warn('Internal Docker URL missing port number - this may cause connection issues'); } } // Allow internal docker URLs in production if (process.env.NODE_ENV === 'production') { - if (!['http:', 'https:', 'http://srv-captain--'].some(prefix => - process.env.WEBHOOK_URL?.startsWith(prefix))) { + const isOk = (url.protocol === 'http:' || url.protocol === 'https:') || process.env.WEBHOOK_URL.startsWith('http://srv-captain--'); + if (!isOk) { console.warn('Production WEBHOOK_URL should use http/https or internal docker URL'); } - } else if (!['http:', 'https:'].includes(url.protocol)) { + } else if (!(url.protocol === 'http:' || url.protocol === 'https:')) { throw new Error('WEBHOOK_URL must use http or https protocol'); } } catch (e) {