Use clean user agent string instead of appVersion when userAgentData …

…is unavailable
microsoft · Feb 26, 2024 · 90945ca · 90945ca
1 parent 997abf0
commit 90945ca
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/browser/telemetryReporter.ts b/src/browser/telemetryReporter.ts
@@ -14,7 +14,9 @@ function getBrowserRelease(navigator: Navigator): string {
 		const browser = navigator.userAgentData.brands[navigator.userAgentData.brands.length - 1];
 		return `${navigator.userAgentData.platform} - ${browser?.brand} v${browser?.version}}`;
 	} else {
-		return navigator.appVersion;
+		// clean the user agent using the logic from here:
+		// https://github.com/microsoft/vscode/blob/main/src/vs/workbench/services/telemetry/browser/workbenchCommonProperties.ts#L14C1-L21C2
+		return navigator.userAgent.replace(/(\d+\.\d+)(\.\d+)+/g, "$1");
 	}
 }