Skip to content

Commit

Permalink
Remove context on web
Browse files Browse the repository at this point in the history
  • Loading branch information
lramos15 committed Aug 10, 2021
1 parent 4f4022c commit c7eefca
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 9 deletions.
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "vscode-extension-telemetry",
"description": "A module for first party microsoft extensions to report consistent telemetry.",
"version": "0.2.7",
"version": "0.2.8",
"author": {
"name": "Microsoft Corporation"
},
Expand Down
6 changes: 0 additions & 6 deletions src/browser/telemetryReporter.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
* Copyright (C) Microsoft Corporation. All rights reserved.
*--------------------------------------------------------*/

import * as vscode from "vscode";
import { ApplicationInsights } from "@microsoft/applicationinsights-web";
import { AppenderData, BaseTelemtryReporter, ITelemetryAppender } from "../common/baseTelemetryReporter";

Expand Down Expand Up @@ -31,11 +30,6 @@ class WebAppInsightsAppender implements ITelemetryAppender {
});
this._aiClient.loadAppInsights();

if (vscode && vscode.env) {
this._aiClient.context.user.id = vscode.env.machineId;
this._aiClient.context.session.id = vscode.env.sessionId;
}

// If we cannot access the endpoint this most likely means it's being blocked
// and we should not attempt to send any telemetry.
if (endpointUrl) {
Expand Down

0 comments on commit c7eefca

Please sign in to comment.