fix: Force posthog recording to be disabled outside cloud

The recordings are not enabled by default, but env variable enables opt-in. This changes the logic that opt-in requires also being a cloud deployment.
n8n-io · Jan 17, 2024 · c790ab7 · c790ab7
1 parent 9945701
commit c790ab7
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/cli/src/services/frontend.service.ts b/packages/cli/src/services/frontend.service.ts
@@ -115,9 +115,9 @@ export class FrontendService {
 				apiHost: config.getEnv('diagnostics.config.posthog.apiHost'),
 				apiKey: config.getEnv('diagnostics.config.posthog.apiKey'),
 				autocapture: false,
-				disableSessionRecording: config.getEnv(
-					'diagnostics.config.posthog.disableSessionRecording',
-				),
+				disableSessionRecording:
+					config.getEnv('diagnostics.config.posthog.disableSessionRecording') ||
+					config.getEnv('deployment.type') !== 'cloud',
 				debug: config.getEnv('logs.level') === 'debug',
 			},
 			personalizationSurveyEnabled: