Skip to content

Commit

Permalink
Only include workspace name as part of telemetry (#2883)
Browse files Browse the repository at this point in the history
### Motivation

I made a mistake when adding the workspace as part of the telemetry. I thought it was the workspace name, but it's actually the path to the workspace, which may include things like `/Users/name/...`.

Let's return only the workspace name to fully anonymize the telemetry.
  • Loading branch information
vinistock authored Nov 20, 2024
1 parent 071aa84 commit 3f62b8e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion vscode/src/client.ts
Original file line number Diff line number Diff line change
Expand Up @@ -498,7 +498,7 @@ export default class Client extends LanguageClient implements ClientInterface {
...error.data,
serverVersion: this.serverVersion,
workspace: new vscode.TelemetryTrustedValue(
this.workingDirectory,
path.basename(this.workingDirectory),
),
},
);
Expand Down

0 comments on commit 3f62b8e

Please sign in to comment.