mirror of
https://github.com/AdguardTeam/AdGuardHome.git
synced 2026-03-04 00:01:12 -05:00
DNS Rewrites are ignored/disabled when a persistent client has protection disabled #5642
Labels
No labels
P1: Critical
P2: High
P3: Medium
P4: Low
UI
bug
cannot reproduce
compatibility
dependencies
docker
documentation
duplicate
enhancement
enhancement
external libs
feature request
good first issue
help wanted
infrastructure
invalid
localization
needs investigation
performance
potential-duplicate
question
recurrent
research
snap
waiting for data
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/AdGuardHome#5642
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @rubin110 on GitHub (May 1, 2025).
Prerequisites
I have checked the Wiki and Discussions and found no answer
I have searched other issues and found no duplicates
I want to report a bug and not ask a question or ask for help
I have set up AdGuard Home correctly and configured clients to use it. (Use the Discussions for help with installing and configuring clients.)
Platform (OS and CPU architecture)
Linux, AMD64 (aka x86_64)
Installation
Docker
Setup
On a router, DHCP is handled by AdGuard Home
AdGuard Home version
0.107.61
Action
10.10.10.2rundig example.comand observe outputexample.comto10.10.10.9910.10.10.2rundig example.comand observe output10.10.10.2rundig example.comand observe output10.10.10.2, unchecking "Use global settings"10.10.10.2rundig example.comand observe outputExpected result
Expected results per step:
1 -
digreturns96.7.128.1983 -
digreturns10.10.10.995 -
digreturns96.7.128.1986 -
digreturns10.10.10.99Expectation should match up with #1558, which defines "Disable Protection" as keeping DNS rewrites still active. The difference with the bug I'm writing here is that this behavior doesn't match up with disabling protection per client.
Actual result
Actual results per step:
1 -
digreturns96.7.128.1983 -
digreturns10.10.10.995 -
digreturns96.7.128.1986 -
digreturns96.7.128.198Additional information and/or screenshots
No response
@EugeneOne1 commented on GitHub (May 6, 2025):
@rubin110, hello. In order for DNS rewrites to be applied, the "Block domains using filters and hosts files" checkbox should be selected in either the "General settings" (common for all clients) or client's settings (for a single persistent client). It's expected behavior not to apply DNS rewrites when this setting is disabled.
@rubin110 commented on GitHub (May 6, 2025):
@EugeneOne1 Thanks for the clarification. Can you please explain the behavior that was discussed in #1558, you closed it out Oct 2021? It sounds like from that issue that the expected behavior for the global option to "Disable Protection" should not impact DNS Rewrites, which outlined in my repro steps is how things currently work right now.
If the expectation is that DNS Rewrites should not work for both global protection being disabled, or per client being disabled. I can write a bug for global since that functionality is currently the opposite of what you're describing.
I would like to also open a UI/UX bug regarding not enough information being provided to the user that DNS Rewrites is part of "Protection" and thus will be disabled if protection is disabled either globally or per client.
@TheCataliasTNT2k commented on GitHub (Oct 12, 2025):
I have the same problem:
I use the rewrites for local DNS resolution.
Be cause we have multiple devices in our local network (and no one wants to remember IP addresses), I need a way to give them domains.
I found three solutions:
So please provide a way, to map wildcard DNS entries to IP addresses, without needing a second DNS server while some clients should not be filtered.
As stated by @rubin110 this should be the current behaviour (according to the documentation / other issues), but it isn't. Or not?
@EugeneOne1