From 233683ce1a5a75ba1cf44530c0f768cc089160e5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Iv=C3=A1n=20Ovejero?= Date: Fri, 27 Oct 2023 18:48:21 +0200 Subject: [PATCH] refactor(core): Remove webhook deregistration on shutdown flag (#7543) Removal missing from [original PR](https://github.com/n8n-io/n8n/pull/7515) after initial revert. --- packages/cli/src/config/schema.ts | 20 +------------------- 1 file changed, 1 insertion(+), 19 deletions(-) diff --git a/packages/cli/src/config/schema.ts b/packages/cli/src/config/schema.ts index 2b6a89c0cd..ed87820b95 100644 --- a/packages/cli/src/config/schema.ts +++ b/packages/cli/src/config/schema.ts @@ -95,7 +95,7 @@ export const schema = { ssl: { enabled: { doc: 'If SSL should be enabled. If `ca`, `cert`, or `key` are defined, this will automatically default to true', - format: 'Boolean', + format: Boolean, default: false, env: 'DB_POSTGRESDB_SSL_ENABLED', }, @@ -692,24 +692,6 @@ export const schema = { env: 'N8N_DISABLE_PRODUCTION_MAIN_PROCESS', doc: 'Disable production webhooks from main process. This helps ensures no http traffic load to main process when using webhook-specific processes.', }, - skipWebhooksDeregistrationOnShutdown: { - /** - * Longer explanation: n8n de-registers webhooks on shutdown / deactivation - * and registers on startup / activation. If we skip - * deactivation on shutdown, webhooks will remain active on 3rd party services. - * We don't have to worry about startup as it always - * checks if webhooks already exist. - * If users want to upgrade n8n, it is possible to run - * two instances simultaneously without downtime, similar - * to blue/green deployment. - * WARNING: Trigger nodes (like Cron) will cause duplication - * of work, so be aware when using. - */ - doc: 'Deregister webhooks on external services only when workflows are deactivated.', - format: Boolean, - default: false, - env: 'N8N_SKIP_WEBHOOK_DEREGISTRATION_SHUTDOWN', - }, }, publicApi: {