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

[FR]: detect certain resolvconf /etc/resolv.conf files as systemd-resolved #3026

Closed
DentonGentry opened this issue Oct 8, 2021 · 0 comments · Fixed by #3027
Closed

[FR]: detect certain resolvconf /etc/resolv.conf files as systemd-resolved #3026

DentonGentry opened this issue Oct 8, 2021 · 0 comments · Fixed by #3027
Labels

Comments

@DentonGentry
Copy link
Contributor

We're seen /etc/resolv.conf files like these a few times:

# Dynamic resolv.conf(5) file for glibc resolver(3) generated by resolvconf(8)
#     DO NOT EDIT THIS FILE BY HAND -- YOUR CHANGES WILL BE OVERWRITTEN
# 127.0.0.53 is the systemd-resolved stub resolver.
# run "systemd-resolve --status" to see details about the actual nameservers.

The file was created by resolvconf, and tailscaled decides to handle it using the resolvconf path, but the only nameserver in the file is systemd-resolved. The systemd-resolved handling in tailscaled allows more options, like only adding 100.100.100.100 for example.com.beta.tailscale.net and letting the system handle everything else.

I think we can detect cases like this and use systemd-resolved. It will mean making resolvOwner parse further into the file to see if a later line overrides the first match found.

@DentonGentry DentonGentry changed the title [Bug]: detect certain resolvconf /etc/resolv.conf files as systemd-resolved [FR]: detect certain resolvconf /etc/resolv.conf files as systemd-resolved Oct 8, 2021
DentonGentry added a commit that referenced this issue Oct 8, 2021
There are /etc/resolv.conf files out there where resolvconf wrote
the file but pointed to systemd-resolved as the nameserver.
We're better off handling those as systemd-resolved.

> # Dynamic resolv.conf(5) file for glibc resolver(3) generated by resolvconf(8)
> #     DO NOT EDIT THIS FILE BY HAND -- YOUR CHANGES WILL BE OVERWRITTEN
> # 127.0.0.53 is the systemd-resolved stub resolver.
> # run "systemd-resolve --status" to see details about the actual nameservers.

Fixes #3026
Signed-off-by: Denton Gentry <dgentry@tailscale.com>
@DentonGentry DentonGentry added L1 Very few Likelihood OS-linux T5 Usability Issue type P2 Aggravating Priority level and removed needs-triage labels Oct 8, 2021
DentonGentry added a commit that referenced this issue Oct 8, 2021
There are /etc/resolv.conf files out there where resolvconf wrote
the file but pointed to systemd-resolved as the nameserver.
We're better off handling those as systemd-resolved.

> # Dynamic resolv.conf(5) file for glibc resolver(3) generated by resolvconf(8)
> #     DO NOT EDIT THIS FILE BY HAND -- YOUR CHANGES WILL BE OVERWRITTEN
> # 127.0.0.53 is the systemd-resolved stub resolver.
> # run "systemd-resolve --status" to see details about the actual nameservers.

Fixes #3026
Signed-off-by: Denton Gentry <dgentry@tailscale.com>
DentonGentry added a commit that referenced this issue Oct 18, 2021
There are /etc/resolv.conf files out there where resolvconf wrote
the file but pointed to systemd-resolved as the nameserver.
We're better off handling those as systemd-resolved.

> # Dynamic resolv.conf(5) file for glibc resolver(3) generated by resolvconf(8)
> #     DO NOT EDIT THIS FILE BY HAND -- YOUR CHANGES WILL BE OVERWRITTEN
> # 127.0.0.53 is the systemd-resolved stub resolver.
> # run "systemd-resolve --status" to see details about the actual nameservers.

Fixes #3026
Signed-off-by: Denton Gentry <dgentry@tailscale.com>
DentonGentry added a commit that referenced this issue Oct 27, 2021
There are /etc/resolv.conf files out there where resolvconf wrote
the file but pointed to systemd-resolved as the nameserver.
We're better off handling those as systemd-resolved.

> # Dynamic resolv.conf(5) file for glibc resolver(3) generated by resolvconf(8)
> #     DO NOT EDIT THIS FILE BY HAND -- YOUR CHANGES WILL BE OVERWRITTEN
> # 127.0.0.53 is the systemd-resolved stub resolver.
> # run "systemd-resolve --status" to see details about the actual nameservers.

Fixes #3026
Signed-off-by: Denton Gentry <dgentry@tailscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant