From d58d5ffbf6c1c7aac2f86ce844e48c49aaf18c96 Mon Sep 17 00:00:00 2001 From: Frank Denis Date: Wed, 3 Oct 2018 18:33:10 +0200 Subject: [PATCH] Oh, Google updated their servers, and "body" is not required any more! --- ChangeLog | 2 ++ dnscrypt-proxy/xtransport.go | 1 - 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 9cd854a8..f11497b0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -6,6 +6,8 @@ - DNS options used to be cleared from DNS queries, with the exception of flags and payload sizes. This is not the case any more. - Android builds use a newer NDK, and add compatibility with API 19. + - DoH queries are smaller, since workarounds are not required any more +after Google updated their implementation. * Version 2.0.16 - On Unix-like systems, the server can run as an unprivileged user, diff --git a/dnscrypt-proxy/xtransport.go b/dnscrypt-proxy/xtransport.go index 7f5fc9cb..92cf7fa8 100644 --- a/dnscrypt-proxy/xtransport.go +++ b/dnscrypt-proxy/xtransport.go @@ -268,7 +268,6 @@ func (xTransport *XTransport) DoHQuery(useGet bool, url *url.URL, body []byte, t qs := url.Query() qs.Add("ct", "") encBody := base64.RawURLEncoding.EncodeToString(body) - qs.Add("body", encBody) qs.Add("dns", encBody) if padding != nil { qs.Add("random_padding", *padding)