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.
This commit is contained in:
Iván Ovejero 2023-10-27 18:48:21 +02:00 committed by GitHub
parent 18dcea8c19
commit 233683ce1a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -95,7 +95,7 @@ export const schema = {
ssl: { ssl: {
enabled: { enabled: {
doc: 'If SSL should be enabled. If `ca`, `cert`, or `key` are defined, this will automatically default to true', 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, default: false,
env: 'DB_POSTGRESDB_SSL_ENABLED', env: 'DB_POSTGRESDB_SSL_ENABLED',
}, },
@ -692,24 +692,6 @@ export const schema = {
env: 'N8N_DISABLE_PRODUCTION_MAIN_PROCESS', 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.', 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: { publicApi: {