Merge pull request microsoft#204374 from microsoft/sandy081/fix-telemetry

sent string value in telemetry

Author: sandy081

src/vs/workbench/contrib/telemetry/browser/telemetry.contribution.ts

@@ -281,12 +281,12 @@ class ConfigurationTelemetryContribution extends Disposable implements IWorkbenc
 	/**
 	 * Report value of a setting only if it is an enum, boolean, or number or an array of those.
 	 */
-	private getValueToReport(key: string, target: ConfigurationTarget.USER_LOCAL | ConfigurationTarget.WORKSPACE): any {
+	private getValueToReport(key: string, target: ConfigurationTarget.USER_LOCAL | ConfigurationTarget.WORKSPACE): string | undefined {
 		const schema = this.configurationRegistry.getConfigurationProperties()[key];
 		const inpsectData = this.configurationService.inspect(key);
 		const value = target === ConfigurationTarget.USER_LOCAL ? inpsectData.user?.value : inpsectData.workspace?.value;
 		if (isNumber(value) || isBoolean(value)) {
-			return value;
+			return value.toString();
 		}
 		if (isString(value)) {
 			if (schema?.enum?.includes(value)) {
@@ -296,15 +296,15 @@ class ConfigurationTelemetryContribution extends Disposable implements IWorkbenc
 		}
 		if (Array.isArray(value)) {
 			if (value.every(v => isNumber(v) || isBoolean(v) || (isString(v) && schema?.enum?.includes(v)))) {
-				return value;
+				return JSON.stringify(value);
 			}
 		}
 		return undefined;
 	}
 
 	private reportTelemetry(key: string, target: ConfigurationTarget.USER_LOCAL | ConfigurationTarget.WORKSPACE): void {
 		type UpdatedSettingEvent = {
-			value: any;
+			value: string | undefined;
 			source: string;
 		};
 		const source = ConfigurationTargetToString(target);