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

NOERROR+NODATA response is not cached #3598

Closed
3 tasks done
ZeroClover opened this issue Sep 15, 2021 · 3 comments
Closed
3 tasks done

NOERROR+NODATA response is not cached #3598

ZeroClover opened this issue Sep 15, 2021 · 3 comments
Assignees
Labels
enhancement external libs Issues that require changes in external libraries. P4: Low
Milestone

Comments

@ZeroClover
Copy link

ZeroClover commented Sep 15, 2021

  • I am running the latest version
  • I checked the documentation and found no answer
  • I checked to make sure that this issue has not already been filed

Issue Details

  • Version of AdGuard Home server:
    v0.107.0-b.11
  • How did you install AdGuard Home:
    Snapcraft
  • How did you setup DNS configuration:
    DoH with dnsproxy
  • CPU architecture:
    AMD64
  • Operating system and version:
    Debian 11

Actual Behavior

According to RFC4074:

Suppose that an authoritative server has an A RR but has no AAAA RR for a hostname. Then, the server should return a response to a query for an AAAA RR of the name with the response code (RCODE) being 0 (indicating no error) and with an empty answer section

In this case, AdGuard Home does not cache responses, even when the minimum cache TTL is set.

Expected Behavior

AdGuard Home follows the same settings as NXDOMAIN to cache NOERROR+NODATA

Screenshots

Screenshot:

image

image

Additional Information

This also seems to affect the dnsproxy

@ainar-g ainar-g added enhancement external libs Issues that require changes in external libraries. P4: Low labels Sep 15, 2021
@ainar-g ainar-g added this to the v0.108.0 milestone Sep 15, 2021
@ameshkov
Copy link
Member

@ainar-g legit issue, we'll need to get the TTL from the SOA section in this case.

@EugeneOne1
Copy link
Member

EugeneOne1 commented Apr 8, 2022

@ZeroClover, hello and apologies for a late response. This seems that it has been implemented in #3707. Could you please check it out in any version of AGH since v0.107.0?

@EugeneOne1
Copy link
Member

@ZeroClover, hello again. I'm going to close the issue for now, feel free to reopen it on any problems with the implementation.

@EugeneOne1 EugeneOne1 modified the milestones: v0.107.7, v0.107.0 Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement external libs Issues that require changes in external libraries. P4: Low
Projects
None yet
Development

No branches or pull requests

4 participants