miracle091/dnscrypt-proxy
miracle091/dnscrypt-proxy
1
0
Fork 0
mirror of https://github.com/DNSCrypt/dnscrypt-proxy.git synced 2025-01-01 00:57:32 +01:00
Code Issues Releases Wiki Activity

cleanup: xtransport: move onion check earlier to fail fast

Browse Source
This commit is contained in:
Markus Linnala 2019-10-16 13:17:57 +03:00 committed by Frank Denis
parent df03065eaf
commit 345ec56cab
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
dnscrypt-proxy/xtransport.go
View File

@ -211,6 +211,10 @@ func (xTransport *XTransport) Fetch(method string, url *url.URL, accept string,
url2.RawQuery = qs.Encode() url2.RawQuery = qs.Encode()
url = &url2 url = &url2
} }
host, _ := ExtractHostAndPort(url.Host, 0)
if xTransport.proxyDialer == nil && strings.HasSuffix(host, ".onion") {
return nil, 0, errors.New("Onion service is not reachable without Tor")
}
req := &http.Request{ req := &http.Request{
Method: method, Method: method,
URL: url, URL: url,
@ -223,10 +227,6 @@ func (xTransport *XTransport) Fetch(method string, url *url.URL, accept string,
req.Body = bc req.Body = bc
} }
var err error var err error
host, _ := ExtractHostAndPort(url.Host, 0)
if xTransport.proxyDialer == nil && strings.HasSuffix(host, ".onion") {
return nil, 0, errors.New("Onion service is not reachable without Tor")
}
resolveByProxy := false resolveByProxy := false
if xTransport.proxyDialer != nil || xTransport.httpProxyFunction != nil { if xTransport.proxyDialer != nil || xTransport.httpProxyFunction != nil {
resolveByProxy = true resolveByProxy = true