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

dns: support AXFR queries #14648

Closed
silverwind opened this issue Aug 6, 2017 · 7 comments
Closed

dns: support AXFR queries #14648

silverwind opened this issue Aug 6, 2017 · 7 comments
Labels
dns Issues and PRs related to the dns subsystem. feature request Issues that request new features to be added to Node.js.

Comments

@silverwind
Copy link
Contributor

Now that we support ANY queries (#13137), I think the foundation is laid to support zone transfers and we can likely share most of the code that is used to parse the ANY response and parse the AXFR response with it.

cc: @XadillaX

@silverwind silverwind added dns Issues and PRs related to the dns subsystem. feature request Issues that request new features to be added to Node.js. labels Aug 6, 2017
@XadillaX
Copy link
Contributor

XadillaX commented Aug 6, 2017

The principle of ANY queries is to parse the packet time by time by calling ares_parse_a_reply, ares_parse_aaaa_reply, etc.

But it seems C-ARES does not support to parse AXFR query.

@tniessen
Copy link
Member

tniessen commented Aug 6, 2017

The main difference between "normal" queries and AXFR queries is that AXFR queries can have their answers spread across any number of messages. @XadillaX might be correct, I am not sure whether c-ares supports this 😕

@silverwind
Copy link
Contributor Author

Yeah, looks like c-ares can indeed not parse a AXFR response. Using the adig utility that comes with it, I only see the initial SOA response being parsed, but not the AXFR content:

$ ./adig -t axfr -s 127.0.0.1 domain.com
DNS server returned answer with no data
id: 18388
flags: qr aa rd
opcode: QUERY
rcode: NOERROR
Questions:
  domain.com.   AXFR
Answers:
NS records:
  domain.com. 3600  SOA ...

@bnoordhuis
Copy link
Member

@silverwind Is the conclusion that this isn't possible? If yes, can you close it out?

(Aside: adig -- like c-ares -- indeed doesn't support AXFR, it only recognizes the tag.)

@silverwind
Copy link
Contributor Author

Yes, we can close it, I just don't want it to be forgotten. Let's see what comes out of #14713, otherwise I'll open an issue on the c-ares tracker.

@Trott
Copy link
Member

Trott commented Aug 11, 2019

Yes, we can close it, I just don't want it to be forgotten. Let's see what comes out of #14713, otherwise I'll open an issue on the c-ares tracker.

@silverwind Did you ever open that issue on the c-ares tracker?

@silverwind
Copy link
Contributor Author

No, I don't think so. Nowadays, I just use https://github.com/mafintosh/dns-packet/ for advanced DNS use cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dns Issues and PRs related to the dns subsystem. feature request Issues that request new features to be added to Node.js.
Projects
None yet
Development

No branches or pull requests

5 participants