useHostProxy > isUseHostProxyEnabled() (microsoft/vscode-copilot-rele…

…ase#3821)
microsoft · Jan 16, 2025 · 5f5594d · 5f5594d
1 parent b1f8f2e
commit 5f5594d
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,9 @@
 # Change Log
 Notable changes will be documented here.
 
+## [0.30.0]
+- useHostProxy > isUseHostProxyEnabled() ([microsoft/vscode-copilot-release#3821](https://github.com/microsoft/vscode-copilot-release/issues/3821))
+
 ## [0.29.0]
 - Update to undici 7.2.0 ([microsoft/vscode-proxy-agent#57](https://github.com/microsoft/vscode-proxy-agent/pull/57))
 - Get options from unpatched agents ([microsoft/vscode-proxy-agent#58](https://github.com/microsoft/vscode-proxy-agent/pull/58))

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@vscode/proxy-agent",
-  "version": "0.29.0",
+  "version": "0.30.0",
   "description": "NodeJS http(s) agent implementation for VS Code",
   "main": "out/index.js",
   "types": "out/index.d.ts",

diff --git a/src/index.ts b/src/index.ts
@@ -74,7 +74,7 @@ export interface ProxyAgentParams {
 	log: Log;
 	getLogLevel(): LogLevel;
 	proxyResolveTelemetry(event: ProxyResolveEvent): void;
-	useHostProxy: boolean;
+	isUseHostProxyEnabled: () => boolean;
 	env: NodeJS.ProcessEnv;
 }
 
@@ -207,7 +207,7 @@ export function createProxyResolver(params: ProxyAgentParams) {
 			return;
 		}
 
-		if (!params.useHostProxy) {
+		if (!params.isUseHostProxyEnabled()) {
 			callback('DIRECT');
 			log.debug('ProxyResolver#resolveProxy unconfigured', url, 'DIRECT', stackText);
 			return;

diff --git a/tests/test-client/src/utils.ts b/tests/test-client/src/utils.ts
@@ -24,7 +24,7 @@ export const directProxyAgentParams: vpa.ProxyAgentParams = {
 	log: console,
 	getLogLevel: () => vpa.LogLevel.Trace,
 	proxyResolveTelemetry: () => undefined,
-	useHostProxy: true,
+	isUseHostProxyEnabled: () => true,
 	loadAdditionalCertificates: async () => [
 		...await loadSystemCertificates({ log: console }),
 		...ca,