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

Support for RFC9606 RRType 261 (RESINFO) #14983

Open
mdavids opened this issue Dec 17, 2024 · 3 comments
Open

Support for RFC9606 RRType 261 (RESINFO) #14983

mdavids opened this issue Dec 17, 2024 · 3 comments

Comments

@mdavids
Copy link

mdavids commented Dec 17, 2024

  • Program: dnsdist
  • Issue type: Feature request

Short description

RFC9606 introduces a new RRtype 'RESINFO'. It would be nice if that could be used with 'DNSQType'.

Usecase

Implement RFC9606 support for our public resolver. The RFC says: "DNS client MUST set the Recursion Desired (RD) bit of the query to 0. The DNS client MUST discard the response if the AA flag in the response is set to 0", which implicates that just adding RESINFO-records to a zone file is not the way to go (see https://marc.info/?l=openbsd-tech&m=173368282507730&w=2).

With a Luarule this could however be done, when the RRType is supported in the 'DNSQtype'-function. Looking at the source code, this is not the case yet.

Description

Simply extend the list of RRtypes with RESINFO.

@rgacogne
Copy link
Member

Makes sense to me! Note that you should be able to use the numerical value in most places in the meantime.

@mdavids
Copy link
Author

mdavids commented Dec 18, 2024

Makes sense to me! Note that you should be able to use the numerical value in most places in the meantime.

Please enlighten me; how would that work? It's not DNSQType.261, is it?

@omoerbeek
Copy link
Member

[QTypeRule(qtype)](https://dnsdist.org/reference/selectors.html?highlight=qtyperule#QTypeRule) says it taks an int, so e.g. QTypeRule(261) should do it.

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

3 participants