From 7943c90d5d23613a7c95b22a8ad3ea1c8e4ed051 Mon Sep 17 00:00:00 2001 From: Mauricio Siu <47042324+Siumauricio@users.noreply.github.com> Date: Sun, 5 Oct 2025 12:07:19 -0600 Subject: [PATCH] refactor: enhance middleware removal logic in Traefik configuration --- packages/server/src/utils/traefik/middleware.ts | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/packages/server/src/utils/traefik/middleware.ts b/packages/server/src/utils/traefik/middleware.ts index c61db23c4..907e65afd 100644 --- a/packages/server/src/utils/traefik/middleware.ts +++ b/packages/server/src/utils/traefik/middleware.ts @@ -195,6 +195,10 @@ export const removePathMiddlewares = async ( } } + if (!config) { + return; + } + const { appName } = app; if (config.http?.middlewares) { @@ -205,13 +209,12 @@ export const removePathMiddlewares = async ( delete config.http.middlewares[stripPrefixMiddleware]; } - if (config?.http?.middlewares) { - // traefik will fail to start if the file contains middlewares entry but no middlewares are defined - const hasNoMiddlewares = Object.keys(config.http.middlewares).length === 0; - if (hasNoMiddlewares) { - // if there aren't any middlewares, remove the whole section - delete config.http.middlewares; - } + if ( + config?.http?.middlewares && + Object.keys(config.http.middlewares).length === 0 + ) { + // if there aren't any middlewares, remove the whole section + delete config.http.middlewares; } // // If http section is empty, remove it completely