refactor: remove no longer needed TLS handshake timeout

Since the whole resolve has a context w/ deadline it's no longer needed.
0xERR0R · Dec 6, 2023 · 603d374 · 603d374
1 parent a6654dc
commit 603d374
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/resolver/upstream_resolver.go b/resolver/upstream_resolver.go
@@ -24,9 +24,8 @@ import (
 )
 
 const (
-	dnsContentType             = "application/dns-message"
-	defaultTLSHandshakeTimeout = 5 * time.Second
-	retryAttempts              = 3
+	dnsContentType = "application/dns-message"
+	retryAttempts  = 3
 )
 
 type upstreamConfig struct {
@@ -94,9 +93,8 @@ func createUpstreamClient(cfg upstreamConfig) upstreamClient {
 			userAgent: cfg.UserAgent,
 			client: &http.Client{
 				Transport: &http.Transport{
-					TLSClientConfig:     &tlsConfig,
-					TLSHandshakeTimeout: defaultTLSHandshakeTimeout,
-					ForceAttemptHTTP2:   true,
+					TLSClientConfig:   &tlsConfig,
+					ForceAttemptHTTP2: true,
 				},
 			},
 			host: cfg.Host,