From d11b39282b56b9cf53670efdee4f46c8d6481c9f Mon Sep 17 00:00:00 2001 From: pascal-fischer <32096965+pascal-fischer@users.noreply.github.com> Date: Fri, 17 May 2024 12:59:46 +0200 Subject: [PATCH] Enable namserver deactivation if unresponsive on iOS (#1982) --- client/internal/dns/upstream.go | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/client/internal/dns/upstream.go b/client/internal/dns/upstream.go index cc31559fa..e82c98fbc 100644 --- a/client/internal/dns/upstream.go +++ b/client/internal/dns/upstream.go @@ -5,7 +5,6 @@ import ( "errors" "fmt" "net" - "runtime" "sync" "sync/atomic" "time" @@ -260,13 +259,10 @@ func (u *upstreamResolverBase) disable(err error) { return } - // todo test the deactivation logic, it seems to affect the client - if runtime.GOOS != "ios" { - log.Warnf("Upstream resolving is Disabled for %v", reactivatePeriod) - u.deactivate(err) - u.disabled = true - go u.waitUntilResponse() - } + log.Warnf("Upstream resolving is Disabled for %v", reactivatePeriod) + u.deactivate(err) + u.disabled = true + go u.waitUntilResponse() } func (u *upstreamResolverBase) testNameserver(server string) error {