summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorOskar Nyberg <oskar@mullvad.net>2022-01-13 12:26:39 +0100
committerOskar Nyberg <oskar@mullvad.net>2022-01-13 12:26:39 +0100
commit251f799a0d557e3753af4b00b505f1385dccfdf7 (patch)
treeaaff187571d359d38afeb60e9b1b997bc2f3bb65
parent12258e4474bf845e5eb900cf48df97d0dfa5a9f2 (diff)
parentbd75b32323d282ea8f2b919d711f911e42157238 (diff)
downloadmullvadvpn-251f799a0d557e3753af4b00b505f1385dccfdf7.tar.xz
mullvadvpn-251f799a0d557e3753af4b00b505f1385dccfdf7.zip
Merge branch 'disable-grpc-http-proxy'
-rw-r--r--CHANGELOG.md2
-rw-r--r--gui/src/main/daemon-rpc.ts3
2 files changed, 5 insertions, 0 deletions
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 83564bc28c..1912b28ebf 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -71,6 +71,8 @@ Line wrap the file at 100 chars. Th
### Fixed
- Always kill `sslocal` if the tunnel monitor fails to start when using bridges.
- Show relay location constraint correctly in the CLI when it is set to `any`.
+- Prevent gRPC from trying to run the app-daemon IPC communication through a HTTP proxy when the
+ environment variable `http_proxy` is set. This caused the app to fail to connect to the daemon.
#### macOS
- Disable built-in DNS resolver in Electron. Prevents Electron from establishing connections to
diff --git a/gui/src/main/daemon-rpc.ts b/gui/src/main/daemon-rpc.ts
index 32fc7adccf..c8cad6fa6a 100644
--- a/gui/src/main/daemon-rpc.ts
+++ b/gui/src/main/daemon-rpc.ts
@@ -584,6 +584,9 @@ export class DaemonRpc {
'grpc.initial_reconnect_backoff_ms': 3000,
'grpc.keepalive_time_ms': Math.pow(2, 30),
'grpc.keepalive_timeout_ms': Math.pow(2, 30),
+ // Prevents grpc-js from parsing the `http_proxy` environment variable and trying to use it
+ // even for IPC sockets.
+ 'grpc.enable_http_proxy': 0,
};
/* eslint-enable @typescript-eslint/naming-convention */
}