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

Adguard Home dashboard should resolve client names (easier) #1963

Closed
Tiscan opened this issue Jul 26, 2020 · 5 comments
Closed

Adguard Home dashboard should resolve client names (easier) #1963

Tiscan opened this issue Jul 26, 2020 · 5 comments
Labels
duplicate Duplicate or merged issues.

Comments

@Tiscan
Copy link

Tiscan commented Jul 26, 2020

Prerequisites

Please answer the following questions for yourself before submitting an issue. YOU MAY DELETE THE PREREQUISITES SECTION.

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

Problem Description

I have a Upstream DNS server entry like:

[/local/lan/51.10.in-addr.arpa/]10.51.1.1

This does allow me to see client names in the dashboard.

This took me a while to figure out. However this isn't very straight forward and isn't required in other software like this.

Proposed Solution

Still think this would be a good FR to simplify... perhaps have the subnet included like:

[/local/lan/10.51.1.0/]10.51.1.1

Automatically parse the reverse resolve?

Also I don't think the above works for IPV6 but not sure it can.

@Tiscan Tiscan changed the title Adguard Home dashboard should resolve client names Adguard Home dashboard should resolve client names (easier) Jul 27, 2020
@szolin
Copy link
Contributor

szolin commented Jul 27, 2020

[/local/lan/10.51.1.0/]10.51.1.1

10.51.1.0 is not a valid domain name here, so this doesn't follow the correct syntax.

However, in

[/local/lan/51.10.in-addr.arpa/]10.51.1.1

51.10.in-addr.arpa is the correct domain name suffix which includes 10.51.0.0/24 for PTR requests.

We should come up with another, more clear specification for this. Maybe to use another special character or something.

@Tiscan
Copy link
Author

Tiscan commented Jul 27, 2020

Yep that is my point.

It would be great if there was a more straight forward way to do this. I would bet that most installs don't have reverse lookup enabled because of how "complex" this is. I realize it's not complex for all folks but most don't have this level of understanding.

@ameshkov
Copy link
Member

We should come up with another, more clear specification for this. Maybe to use another special character or something.

Maybe some setting that allows specifying LAN DNS server?

@Tiscan
Copy link
Author

Tiscan commented Aug 2, 2020

Also for those who don't have a LAN DNS server in the DHCP settings for Adguard it would be nice to be able to specify a local search domain but this is covered in issue #1956.

@ameshkov
Copy link
Member

Merged with #2704

@ameshkov ameshkov added duplicate Duplicate or merged issues. and removed feature request labels Feb 18, 2021
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

3 participants