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

dnsdist: Ponder what to do about header-only queries #9961

Open
rgacogne opened this issue Jan 12, 2021 · 1 comment
Open

dnsdist: Ponder what to do about header-only queries #9961

rgacogne opened this issue Jan 12, 2021 · 1 comment

Comments

@rgacogne
Copy link
Member

rgacogne commented Jan 12, 2021

  • Program: dnsdist
  • Issue type: Bug report

Short description

As reported by Arnaud Gavara on the mailing-list 1, dnsdist currently drops queries that have a qdcount of zero. That used to be fine (it was used by rfc7873 but it seems that nobody cared about it) but now that it is described as a conformance test in rfc8906 we might need to think about how to handle that.
The issue is that a lot of our rules, and likely a lot of custom Lua code, expect that a query has a qname, a qtype and a qclass, and I'm not sure how to deal with that in a compatible way. I don't want to break setups on an upgrade, even a major one, if I don't have to.

A good start might be to reply with FORMERR or NOTIMP instead of dropping the packet, at least until we can figure out a way to actually pass that query to the backend.

@rgacogne
Copy link
Member Author

Moving to the 1.7 milestone for a way to pass the query to the backends.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant