dnschef icon indicating copy to clipboard operation
dnschef copied to clipboard

Add support for HTTPS records

Open jkl-caliber opened this issue 3 years ago • 4 comments

HTTPS is a new DNS record type with increasing support.

https://blog.cloudflare.com/speeding-up-https-and-http-3-negotiation-with-dns/

This patch adds support to dnschef. The associated PR has been filed against dnslib. The new functionality in dnschef depends on that new code.

https://github.com/paulc/dnslib/pull/33

jkl-caliber avatar Feb 03 '22 21:02 jkl-caliber

I think this needs more code to work? When trying to use the code in this PR I get the following error:

          _                _          __
         | | version 0.4  | |        / _|
       __| |_ __  ___  ___| |__   ___| |_
      / _` | '_ \/ __|/ __| '_ \ / _ \  _|
     | (_| | | | \__ \ (__| | | |  __/ |
      \__,_|_| |_|___/\___|_| |_|\___|_|
                   [email protected]

(15:24:49) [*] DNSChef started on interface: 127.0.0.1
(15:24:49) [*] Using the following nameservers: 8.8.8.8
(15:24:49) [*] Cooking A replies for domain testflight.apple.com with '192.168.8.221'
WARNING: DNS Record 'HTTPS' is not supported. Ignoring section contents.

andyacer avatar Jun 16 '22 19:06 andyacer

Are you using a patched copy of dnslib? https://github.com/paulc/dnslib/pull/35

jkl-caliber avatar Jun 16 '22 22:06 jkl-caliber

ah sorry, no, I'd assumed that dnslib had incorporated the PR you mentioned. I'll try that, thanks!

andyacer avatar Jun 16 '22 23:06 andyacer

Followup - dnslib merged in your other PR - dnschef should merge this one too =) It's working great now that dnslib is updated, thanks for submitting both of those PRs.

andyacer avatar Jul 26 '22 22:07 andyacer