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 Rewrites Hierarchy broken since version 0.107.37 #6253

Closed
4 tasks done
mikurth opened this issue Sep 23, 2023 · 2 comments
Closed
4 tasks done

DNS Rewrites Hierarchy broken since version 0.107.37 #6253

mikurth opened this issue Sep 23, 2023 · 2 comments
Labels
duplicate Duplicate or merged issues.

Comments

@mikurth
Copy link

mikurth commented Sep 23, 2023

Prerequisites

Platform (OS and CPU architecture)

Linux, ARM64

Installation

Docker

Setup

On one machine

AdGuard Home version

v0.107.37

Action

nslookup -debug -type=a 'XXX.ZZZ.YYY.com' '$YOUR_AGH_ADDRESS'

Server: 172.22.1.3
Address: 172.22.1.3#53


QUESTIONS:
    XXX.ZZZ.YYY.com, type = A, class = IN
ANSWERS:
->  XXX.ZZZ.YYY.com
    internet address = 172.22.1.11
    ttl = 10
AUTHORITY RECORDS:
ADDITIONAL RECORDS:

Non-authoritative answer:
Name: XXX.ZZZ.YYY.com
Address: 172.22.1.11

Expected result

response with ip 172.22.1.10

Actual result

response with ip 172.22.1.11

Additional information and/or screenshots

Since version 107.37 the DNS overwrite hierarchy is no longer correct. I have two overwrites defined:

rewrites: - domain: '*.YYY.com' answer: 172.22.1.11 - domain: '*.ZZZ.YYY.com' answer: 172.22.1.10

Until version 0.107.36 (I just tested it out) a dns query for XXX.ZZZ.YYY.pro returned the 172.22.1.10 ip however since version 0.107.37 it returns the one from higher level 172.22.1.11.

@EugeneOne1
Copy link
Member

@mikurth, should be fixed on edge, see #6226.

@EugeneOne1 EugeneOne1 closed this as not planned Won't fix, can't repro, duplicate, stale Sep 23, 2023
@EugeneOne1 EugeneOne1 added the duplicate Duplicate or merged issues. label Sep 23, 2023
@mikurth
Copy link
Author

mikurth commented Sep 23, 2023

perfect thanks. Sorry missed that one..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate Duplicate or merged issues.
Projects
None yet
Development

No branches or pull requests

2 participants