Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TooMuchData errors #19

Closed
vbauerster opened this issue Oct 25, 2019 · 9 comments
Closed

TooMuchData errors #19

vbauerster opened this issue Oct 25, 2019 · 9 comments
Assignees
Labels
bug Something isn't working

Comments

@vbauerster
Copy link

Hello!

I tried doh-client and got following errors:

ERROR [doh_client::http2] GetResponse: DNS parser error: TooMuchData
@LinkTed
Copy link
Owner

LinkTed commented Oct 26, 2019

Hi,
this error appears when the DoH server responds with a content bigger than 512 bytes. According to the RFC 1035, a DNS packet has to be 512 bytes or less. Can you tell me which DoH server you use?

@vbauerster
Copy link
Author

I'm using unbound as front forwarding recursive queries to doh-client with a-and-a.

@LinkTed
Copy link
Owner

LinkTed commented Oct 26, 2019

Can you give me a example DNS request, where this error appears. I use the following argument:

doh-client /etc/ca-certificates/extracted/tls-ca-bundle.pem -vv --remote-addr=217.169.20.23:443 --domain dns.aa.net.uk

@vbauerster
Copy link
Author

Unfortunately I can't narrow which query is causing that, but logs suggest apparently all:
Screenshot-2019-10-26-at-23-09-12.png

@LinkTed
Copy link
Owner

LinkTed commented Oct 26, 2019

I add more debug output to the code to get the request and corresponded response. Can you use the fc18f07 commit and use the -vvv argument, please?

@vbauerster
Copy link
Author

doh-client.log.gz

@LinkTed
Copy link
Owner

LinkTed commented Oct 27, 2019

According to the RFC 6891 the maximum payload of dns packet, was increased. So i increased the size. Can you test the doh-client with the commit 72fad86 and tell me if it works? By the way, if you use unbound to cache the dns packet then you should deactivate the cache mechanism of the doh-client by using the argument -c 0.

@LinkTed LinkTed self-assigned this Oct 27, 2019
@LinkTed LinkTed added the bug Something isn't working label Oct 27, 2019
@vbauerster
Copy link
Author

I confirm, that it works now. Thank you for quick fix!
There are public resolvers with dns stamps available. It would be nice if doh-client would support dns stamp format as source for a resolver. For example I had to use Online DNS Stamp calculator first, in order to extract domain and ip from a stamp like sdns://AgcAAAAAAAAADTIxNy4xNjkuMjAuMjIgPhoaD2xT8-l6SS1XCEtbmAcFnuBXqxUFh2_YP9o9uDgNZG5zLmFhLm5ldC51awovZG5zLXF1ZXJ5.

@LinkTed
Copy link
Owner

LinkTed commented Oct 27, 2019

Okay, thank you for the reporting.

@LinkTed LinkTed closed this as completed Oct 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants