From 8663bbf1a8c90a6d73a0679b0ee3163ef1bdee75 Mon Sep 17 00:00:00 2001
From: "borja (aider)" <borja@brobert.net>
Date: Thu, 27 Mar 2025 20:31:22 +0100
Subject: [PATCH] fix: skip webhook self-test for internal production URLs

---
 src/services/webhook-manager.ts | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/services/webhook-manager.ts b/src/services/webhook-manager.ts
index 2654c06..a39ba7c 100644
--- a/src/services/webhook-manager.ts
+++ b/src/services/webhook-manager.ts
@@ -176,6 +176,13 @@ export class WebhookManager {
 				throw new Error('WEBHOOK_URL is not set');
 			}
 
+			// Skip self-test in production if using internal URL
+			if (process.env.NODE_ENV === 'production' && 
+				process.env.WEBHOOK_URL.startsWith('http://srv-captain--')) {
+				console.log('ℹ️ Skipping self-test for internal production URL');
+				return true;
+			}
+
 			console.log('ℹ️ Testing webhook endpoint:', process.env.WEBHOOK_URL);
 			
 			const testPayload = {
@@ -189,6 +196,7 @@ export class WebhookManager {
 					'Content-Type': 'application/json',
 				},
 				body: JSON.stringify(testPayload),
+				timeout: 5000 // Add timeout to prevent hanging
 			});
 
 			if (!response.ok) {