Merge pull request #209 from Tantalor93/fixresolution

do not pre resolve server hostnames to mitigate cert validation errors

cmd/benchmark.go

@@ -199,18 +199,6 @@ func (b *Benchmark) prepare() error {
 
 	b.addPortIfMissing()
 
-	if !b.useDoH && !b.useQuic {
-		host, port, err := net.SplitHostPort(b.Server)
-		if err != nil {
-			return errors.New("failed to parse --server")
-		}
-		addr, err := net.ResolveIPAddr("ip", host)
-		if err != nil {
-			return errors.New("failed to resolve --server address")
-		}
-		b.Server = net.JoinHostPort(addr.String(), port)
-	}
-
 	if b.Count == 0 && b.Duration == 0 {
 		b.Count = 1
 	}

cmd/benchmark_test.go

@@ -638,21 +638,6 @@ func TestBenchmark_prepare(t *testing.T) {
 			benchmark:  Benchmark{Server: "fddd:dddd::"},
 			wantServer: "[fddd:dddd::]:53",
 		},
-		{
-			name:       "server - resolve addr for plain DNS",
-			benchmark:  Benchmark{Server: "google-public-dns-a.google.com"},
-			wantServer: "8.8.8.8:53",
-		},
-		{
-			name:       "server - resolve addr with port for plain DNS",
-			benchmark:  Benchmark{Server: "google-public-dns-a.google.com:53"},
-			wantServer: "8.8.8.8:53",
-		},
-		{
-			name:       "server - resolve addr for DoT",
-			benchmark:  Benchmark{Server: "google-public-dns-a.google.com", DOT: true},
-			wantServer: "8.8.8.8:853",
-		},
 		{
 			name:       "server - DoT with IP address",
 			benchmark:  Benchmark{Server: "8.8.8.8", DOT: true},