From dedcdbd31496928d3cb8820fd0c2542d056bf7de Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E0=A4=95=E0=A4=BE=E0=A4=B0=E0=A4=A4=E0=A5=8B=E0=A4=AB?= =?UTF-8?q?=E0=A5=8D=E0=A4=AB=E0=A5=87=E0=A4=B2=E0=A4=B8=E0=A5=8D=E0=A4=95?= =?UTF-8?q?=E0=A5=8D=E0=A4=B0=E0=A4=BF=E0=A4=AA=E0=A5=8D=E0=A4=9F=E2=84=A2?= Date: Thu, 13 Mar 2025 18:08:37 +0100 Subject: [PATCH] fix(core): License should ignore empty input on onFeatureChange (#13912) --- packages/cli/src/license.ts | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/packages/cli/src/license.ts b/packages/cli/src/license.ts index 1f2ea7da56..1b7da1d51c 100644 --- a/packages/cli/src/license.ts +++ b/packages/cli/src/license.ts @@ -131,11 +131,18 @@ export class License { } async onFeatureChange(_features: TFeatures): Promise { + const { isMultiMain, isLeader } = this.instanceSettings; + + if (Object.keys(_features).length === 0) { + this.logger.error('Empty license features recieved', { isMultiMain, isLeader }); + return; + } + this.logger.debug('License feature change detected', _features); this.checkIsLicensedForMultiMain(_features); - if (this.instanceSettings.isMultiMain && !this.instanceSettings.isLeader) { + if (isMultiMain && !isLeader) { this.logger .scoped(['scaling', 'multi-main-setup', 'license']) .debug('Instance is not leader, skipping sending of "reload-license" command...');