Disabled telemetry.

microsoft · Mar 14, 2019 · df3a3f6 · df3a3f6
1 parent 3ad8b8c
commit df3a3f6
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/packages/app/main/src/telemetry/telemetryService.spec.ts b/packages/app/main/src/telemetry/telemetryService.spec.ts
@@ -124,6 +124,8 @@ describe('TelemetryService', () => {
     expect(mockAITrackEvent).not.toHaveBeenCalled();
   });
 
+  // NOTE: Disabled for v4.3
+  /*
   it('should track events', () => {
     const mockStartup = jest.fn(() => null);
     (TelemetryService as any).startup = mockStartup;
@@ -145,4 +147,5 @@ describe('TelemetryService', () => {
       properties: { some: 'property' },
     });
   });
+  */
 });
diff --git a/packages/app/main/src/telemetry/telemetryService.ts b/packages/app/main/src/telemetry/telemetryService.ts
@@ -42,6 +42,15 @@ export class TelemetryService {
   private static _client: AppInsights.TelemetryClient;
   private static _hasStarted: boolean = false;
 
+  public static trackEvent(..._args: any[]): void {
+    // no-op until telemetry is re-enabled
+    if (this._client || this.enabled || this.startup) {
+      return;
+    }
+  }
+
+  // NOTE: Disabled for v4.3
+  /*
   public static trackEvent(name: string, properties?: { [key: string]: any }): void {
     if (!this.enabled || !name) {
       return;
@@ -55,7 +64,7 @@ export class TelemetryService {
       // swallow the exception; we don't want to crash the app
       // on a failed attempt to collect usage data
     }
-  }
+  }*/
 
   private static get enabled(): boolean {
     const settings: SettingsImpl = getSettings() || ({} as SettingsImpl);