maybe

PostHog · Sep 5, 2024 · ad1e9f0 · ad1e9f0
1 parent c76f65e
commit ad1e9f0
Showing 1 changed file with 5 additions and 7 deletions.
diff --git 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<void> {