From ac65cc97f4e41cbfa0f99bb32d304668de0fe41c Mon Sep 17 00:00:00 2001 From: Mauricio Siu Date: Sun, 5 Apr 2026 14:08:55 -0600 Subject: [PATCH] fix: surface actual error message in ntfy test connection The catch block was swallowing the real error from the ntfy server, making it impossible to diagnose connection failures (e.g. SSL, DNS, auth issues). Now the underlying error message is included in the tRPC error response. Closes #4047 --- apps/dokploy/server/api/routers/notification.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/apps/dokploy/server/api/routers/notification.ts b/apps/dokploy/server/api/routers/notification.ts index 8d8ddd9de..dd96bc302 100644 --- a/apps/dokploy/server/api/routers/notification.ts +++ b/apps/dokploy/server/api/routers/notification.ts @@ -677,7 +677,10 @@ export const notificationRouter = createTRPCRouter({ } catch (error) { throw new TRPCError({ code: "BAD_REQUEST", - message: "Error testing the notification", + message: + error instanceof Error + ? `Error testing the notification: ${error.message}` + : "Error testing the notification", cause: error, }); }