[wip] Hash newly added data

Still need to hash the file path. Could try to hash the lock path and use leaf for file name unhashed or set them as properties at the end with some unhashed hardvalue like dir ; hashed. file: hashed

Author: nagilson

vscode-dotnet-runtime-library/src/EventStream/EventStreamEvents.ts

@@ -7,6 +7,7 @@ import * as path from 'path';
 import { IDotnetInstallationContext } from '../Acquisition/IDotnetInstallationContext';
 import { EventType } from './EventType';
 import { IEvent } from './IEvent';
+import { TelemetryUtilities } from './FileUtilities';
 
 // tslint:disable max-classes-per-file
 
@@ -19,7 +20,7 @@ export class DotnetAcquisitionStarted extends IEvent {
     }
 
     public getProperties() {
-        return {AcquisitionStartVersion : this.version, extensionId : this.requestingExtensionId};
+        return {AcquisitionStartVersion : this.version, extensionId : TelemetryUtilities.HashData(this.requestingExtensionId)};
     }
 }
 
@@ -32,7 +33,7 @@ export class DotnetRuntimeAcquisitionStarted extends IEvent {
     }
 
     public getProperties() {
-        return {extensionId : this.requestingExtensionId};
+        return {extensionId : TelemetryUtilities.HashData(this.requestingExtensionId)};
     }
 }
 
@@ -45,7 +46,7 @@ export class DotnetSDKAcquisitionStarted extends IEvent {
     }
 
     public getProperties() {
-        return {extensionId : this.requestingExtensionId};
+        return {extensionId : TelemetryUtilities.HashData(this.requestingExtensionId)};
     }
 }
 
@@ -313,7 +314,8 @@ export class DotnetAcquisitionInProgress extends IEvent {
     constructor(public readonly version: string, public readonly requestingExtensionId: string | null) { super(); }
 
     public getProperties() {
-        return {InProgressInstallationVersion : this.version, extensionId : this.requestingExtensionId != null ? this.requestingExtensionId : ''};
+        return {InProgressInstallationVersion : this.version, extensionId : this.requestingExtensionId != null ?
+            TelemetryUtilities.HashData(this.requestingExtensionId) : ''};
     }
 }
 
@@ -324,7 +326,8 @@ export class DotnetAcquisitionAlreadyInstalled extends IEvent {
     constructor(public readonly version: string, public readonly requestingExtensionId: string | null) { super(); }
 
     public getProperties() {
-        return {AlreadyInstalledVersion : this.version, extensionId : this.requestingExtensionId != null ? this.requestingExtensionId : ''};
+        return {AlreadyInstalledVersion : this.version, extensionId : this.requestingExtensionId != null ?
+            TelemetryUtilities.HashData(this.requestingExtensionId) : ''};
     }
 }
 
@@ -362,7 +365,7 @@ export class DotnetAcquisitionRequested extends DotnetAcquisitionMessage {
 
     public getProperties() {
         return {AcquisitionStartVersion : this.version,
-                RequestingExtensionId: this.requestingId};
+                RequestingExtensionId: TelemetryUtilities.HashData(this.requestingId)};
     }
 }
 
@@ -376,7 +379,7 @@ export class DotnetAcquisitionStatusRequested extends DotnetAcquisitionMessage {
 
     public getProperties() {
         return {AcquisitionStartVersion : this.version,
-                RequestingExtensionId: this.requestingId};
+                RequestingExtensionId: TelemetryUtilities.HashData(this.requestingId)};
     }
 }
 

vscode-dotnet-runtime-library/src/EventStream/LoggingObserver.ts

@@ -4,7 +4,7 @@
  * ------------------------------------------------------------------------------------------ */
 import * as fs from 'fs';
 import { IEvent } from './IEvent';
-import { ILoggingObserver } from './ILoggingObserver';
+import { ILoggingObserver } from '../test/ILoggingObserver';
 
 export class LoggingObserver implements ILoggingObserver {
     private log: string[] = [];

vscode-dotnet-runtime-library/src/EventStream/TelemetryUtilities.ts

@@ -0,0 +1,17 @@
+ /* --------------------------------------------------------------------------------------------
+ * Copyright (c) Microsoft Corporation. All rights reserved.
+ * Licensed under the MIT License. See License.txt in the project root for license information.
+ * ------------------------------------------------------------------------------------------ */
+ import * as crypto from 'crypto';
+import { TextEncoder } from 'util';
+
+export class TelemetryUtilities
+{
+    public static HashData(dataToHash: string) : string
+    {
+        const hasher = crypto.createHash('sha256');
+        const utf8Bytes = new TextEncoder().encode(dataToHash.toUpperCase());
+        const hashedData = hasher.update(utf8Bytes).digest('hex').toLowerCase();
+        return hashedData;
+    }
+}

vscode-dotnet-runtime-library/src/Utils/IIssueContext.ts

@@ -3,7 +3,7 @@
  * Licensed under the MIT License. See License.txt in the project root for license information.
  * ------------------------------------------------------------------------------------------ */
 import { IEventStream } from '../EventStream/EventStream';
-import { ILoggingObserver } from '../EventStream/ILoggingObserver';
+import { ILoggingObserver } from '../test/ILoggingObserver';
 import { IWindowDisplayWorker } from '../EventStream/IWindowDisplayWorker';
 import { ErrorConfiguration } from './ErrorHandler';
 import { IExtensionConfigurationWorker } from './IExtensionConfigurationWorker';

vscode-dotnet-runtime-library/src/test/ILoggingObserver.ts

@@ -0,0 +1,12 @@
+/* --------------------------------------------------------------------------------------------
+ * Copyright (c) Microsoft Corporation. All rights reserved.
+ * Licensed under the MIT License. See License.txt in the project root for license information.
+ * ------------------------------------------------------------------------------------------ */
+import { IEvent } from '../EventStream/IEvent';
+import { IEventStreamObserver } from '../EventStream/IEventStreamObserver';
+
+export interface ILoggingObserver extends IEventStreamObserver {
+    post(event: IEvent): void;
+    dispose(): void;
+    getFileLocation(): string;
+}

vscode-dotnet-runtime-library/src/test/mocks/MockObjects.ts

@@ -12,7 +12,7 @@ import { VersionResolver } from '../../Acquisition/VersionResolver';
 import { IEventStream } from '../../EventStream/EventStream';
 import { DotnetAcquisitionCompleted, TestAcquireCalled } from '../../EventStream/EventStreamEvents';
 import { IEvent } from '../../EventStream/IEvent';
-import { ILoggingObserver } from '../../EventStream/ILoggingObserver';
+import { ILoggingObserver } from '../ILoggingObserver';
 import { ITelemetryReporter } from '../../EventStream/TelemetryObserver';
 import { IExistingPath, IExtensionConfiguration } from '../../IExtensionContext';
 import { IExtensionState } from '../../IExtensionState';