fix: include url on HTTP error (#29)

lidel · web-flow · commit 08689e9d0b8a · 2025-01-08T21:47:36.000+01:00
we have per TLD resolvers and this helps end user
to self-service when one of DoH resolvers goes down
diff --git a/request.go b/request.go
@@ -42,7 +42,7 @@ func doRequest(ctx context.Context, url string, m *dns.Msg) (*dns.Msg, error) {
 	defer resp.Body.Close()
 
 	if resp.StatusCode != http.StatusOK {
-		return nil, fmt.Errorf("HTTP error: %q [%d]", resp.Status, resp.StatusCode)
+		return nil, fmt.Errorf("HTTP error: %q [%d] from %q", resp.Status, resp.StatusCode, url)
 	}
 
 	if ct := resp.Header.Get("Content-Type"); ct != dohMimeType {

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ func doRequest(ctx context.Context, url string, m dns.Msg) (dns.Msg, error) {`
`42`	`42`	`defer resp.Body.Close()`
`43`	`43`
`44`	`44`	`if resp.StatusCode != http.StatusOK {`
`45`		`- return nil, fmt.Errorf("HTTP error: %q [%d]", resp.Status, resp.StatusCode)`
	`45`	`+ return nil, fmt.Errorf("HTTP error: %q [%d] from %q", resp.Status, resp.StatusCode, url)`
`46`	`46`	`}`
`47`	`47`
`48`	`48`	`if ct := resp.Header.Get("Content-Type"); ct != dohMimeType {`