From 03dea4713071b4f616ff83d4d54e421036734c59 Mon Sep 17 00:00:00 2001
From: William Elwood <w.elwood08@gmail.com>
Date: Wed, 30 Oct 2019 01:33:20 +0000
Subject: [PATCH] Remove dead code paths

These paths were unreachable because XTransport.Get already checks the same conditions.
---
 dnscrypt-proxy/sources.go | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/dnscrypt-proxy/sources.go b/dnscrypt-proxy/sources.go
index 60c95d14..352b5664 100644
--- a/dnscrypt-proxy/sources.go
+++ b/dnscrypt-proxy/sources.go
@@ -1,7 +1,6 @@
 package main
 
 import (
-	"errors"
 	"fmt"
 	"io"
 	"io/ioutil"
@@ -95,13 +94,7 @@ func fetchWithCache(xTransport *XTransport, urlStr string, cacheFile string, ref
 		return
 	}
 	resp, _, err = xTransport.Get(url, "", 30*time.Second)
-	if err == nil && resp != nil && (resp.StatusCode < 200 || resp.StatusCode > 299) {
-		err = fmt.Errorf("Webserver returned code %d", resp.StatusCode)
-		return
-	} else if err != nil {
-		return
-	} else if resp == nil {
-		err = errors.New("Webserver returned an error")
+	if err != nil {
 		return
 	}
 	var bin []byte