Skip to content

Commit 7737571

Browse files
committed
Merge pull request #1 from beyondkmp/https
Reprioritize to HTTP->SOCKS->HTTPS & use http:// in HTTPS proxy URL
2 parents 3379a47 + a7d5387 commit 7737571

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/index.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,12 +27,7 @@ export async function getSystemProxy(): Promise<ProxyConfig | undefined> {
2727

2828
const noProxy = proxySettings.ExceptionsList || [];
2929

30-
if (proxySettings.HTTPSEnable && proxySettings.HTTPSProxy && proxySettings.HTTPSPort) {
31-
return {
32-
proxyUrl: `https://${proxySettings.HTTPSProxy}:${proxySettings.HTTPSPort}`,
33-
noProxy
34-
};
35-
} else if (proxySettings.HTTPEnable && proxySettings.HTTPProxy && proxySettings.HTTPPort) {
30+
if (proxySettings.HTTPEnable && proxySettings.HTTPProxy && proxySettings.HTTPPort) {
3631
return {
3732
proxyUrl: `http://${proxySettings.HTTPProxy}:${proxySettings.HTTPPort}`,
3833
noProxy
@@ -42,6 +37,11 @@ export async function getSystemProxy(): Promise<ProxyConfig | undefined> {
4237
proxyUrl: `socks://${proxySettings.SOCKSProxy}:${proxySettings.SOCKSPort}`,
4338
noProxy
4439
};
40+
} else if (proxySettings.HTTPSEnable && proxySettings.HTTPSProxy && proxySettings.HTTPSPort) {
41+
return {
42+
proxyUrl: `http://${proxySettings.HTTPSProxy}:${proxySettings.HTTPSPort}`,
43+
noProxy
44+
};
4545
} else {
4646
return undefined;
4747
}

0 commit comments

Comments
 (0)