From ad1e9f0ee4f9d06ea3ba03380bca0520aae67a81 Mon Sep 17 00:00:00 2001 From: olly Date: Thu, 5 Sep 2024 20:38:37 +0200 Subject: [PATCH] maybe --- .../worker/ingestion/property-definitions-manager.ts | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/plugin-server/src/worker/ingestion/property-definitions-manager.ts b/plugin-server/src/worker/ingestion/property-definitions-manager.ts index d113c741c39a9..36e7102e0bc79 100644 --- a/plugin-server/src/worker/ingestion/property-definitions-manager.ts +++ b/plugin-server/src/worker/ingestion/property-definitions-manager.ts @@ -109,13 +109,11 @@ export class PropertyDefinitionsManager { }) this.propertyDefinitionsCache = new PropertyDefinitionsCache(serverConfig) - this.teamIdsToSkip = new Set( - // I have no idea why this is needed but otherwise I get "" - (serverConfig.SKIP_DEFINITIONS_FOR_TEAM_IDS ?? '') - .trim() - .split(',') - .map((id) => parseInt(id)) - ) + if (serverConfig.SKIP_DEFINITIONS_FOR_TEAM_IDS.length > 0) { + this.teamIdsToSkip = new Set( + serverConfig.SKIP_DEFINITIONS_FOR_TEAM_IDS.split(',').map((id) => parseInt(id)) + ) + } } public async updateEventNamesAndProperties(teamId: number, event: string, properties: Properties): Promise {