From eb49306b801d8d38fdd6515c90f57b37907393ef Mon Sep 17 00:00:00 2001 From: Quentin McGaw Date: Tue, 4 Nov 2025 14:47:24 +0000 Subject: [PATCH] hotfix(health): change default icmp target to 1.1.1.1 - Cloudflare's 1.1.1.1 seems more reliable than the VPN server public IP address you connect to - This can still be changed back to 0.0.0.0 to use the VPN server IP address if needed --- Dockerfile | 2 +- internal/configuration/settings/health.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Dockerfile b/Dockerfile index ffc363d4..c15d7bfa 100644 --- a/Dockerfile +++ b/Dockerfile @@ -164,7 +164,7 @@ ENV VPN_SERVICE_PROVIDER=pia \ # Health HEALTH_SERVER_ADDRESS=127.0.0.1:9999 \ HEALTH_TARGET_ADDRESS=cloudflare.com:443 \ - HEALTH_ICMP_TARGET_IP=0.0.0.0 \ + HEALTH_ICMP_TARGET_IP=1.1.1.1 \ HEALTH_RESTART_VPN=on \ # DNS over TLS DOT=on \ diff --git a/internal/configuration/settings/health.go b/internal/configuration/settings/health.go index ea793d8a..3c8fdf81 100644 --- a/internal/configuration/settings/health.go +++ b/internal/configuration/settings/health.go @@ -29,7 +29,7 @@ type Health struct { // It cannot be the empty string in the internal state. TargetAddress string // ICMPTargetIP is the IP address to use for ICMP echo requests - // in the health checker. It can be set to an unspecified address + // in the health checker. It can be set to an unspecified address (0.0.0.0) // such that the VPN server IP is used, which is also the default behavior. ICMPTargetIP netip.Addr // RestartVPN indicates whether to restart the VPN connection