Microsoft/vscode
1
0
Fork 0
mirror of https://github.com/microsoft/vscode.git synced 2026-05-17 13:50:46 +01:00
Code Issues Packages Projects Releases Wiki Activity

Add telemetry on the number of additional certificates

Browse Source
This commit is contained in:
Christof Marti
2026-01-14 11:54:23 +01:00
parent 5f56685431
commit ff6cd330c8
1 changed files with 24 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/vs/workbench/api/node/proxyResolver.ts
+24 -1
View File
@@ -105,7 +105,14 @@ export function connectProxyResolver(
extHostLogService.trace('ProxyResolver#loadAdditionalCertificates: Loading test certificates');
promises.push(Promise.resolve(https.globalAgent.testCertificates as string[]));
}
return (await Promise.all(promises)).flat();
const result = (await Promise.all(promises)).flat();
mainThreadTelemetry.$publicLog2<AdditionalCertificatesEvent, AdditionalCertificatesClassification>('additionalCertificates', {
count: result.length,
isRemote,
loadLocalCertificates,
useNodeSystemCerts,
});
return result;
},
env: process.env,
};
@@ -257,6 +264,22 @@ function recordFetchFeatureUse(mainThreadTelemetry: MainThreadTelemetryShape, fe
}
}
type AdditionalCertificatesClassification = {
owner: 'chrmarti';
comment: 'Tracks the number of additional certificates loaded for TLS connections';
count: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'Number of additional certificates loaded' };
isRemote: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'Whether this is a remote extension host' };
loadLocalCertificates: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'Whether local certificates are loaded' };
useNodeSystemCerts: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'Whether Node.js system certificates are used' };
};
type AdditionalCertificatesEvent = {
count: number;
isRemote: boolean;
loadLocalCertificates: boolean;
useNodeSystemCerts: boolean;
};
type ProxyResolveStatsClassification = {
owner: 'chrmarti';
comment: 'Performance statistics for proxy resolution';