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

fix: use default (google) dns servers if system resolv.conf is invalid #447

Merged
merged 2 commits into from
Jul 29, 2024

Conversation

dav1do
Copy link
Contributor

@dav1do dav1do commented Jul 26, 2024

If the system dns conf is missing nameservers, dns queries will never be resolved. Currently, hickory-dns doesn't error in this case so we handle it manually until the fix is released. This happens on osx if wifi is disabled when you start c1 and is never able to resolve itself without this change, even after connectivity returns. However, if the interface is active when the process starts and then wifi is disabled, it will resolve itself and we will use whatever was defined in resolv.conf.

@dav1do dav1do requested review from nathanielc and a team as code owners July 26, 2024 22:03
Copy link

linear bot commented Jul 26, 2024

dav1do added 2 commits July 26, 2024 16:14
If the system dns conf is missing nameservers, dns queries will never be resolved. Currently, hickory-dns doesn't error in this case so we handle it manually until the fix is released. This happens on osx if wifi is disabled when you start c1 and is never able to resolve itself without this change.
@dav1do dav1do force-pushed the fix/aes-247-dns-errors branch from cc05665 to 06a8f64 Compare July 26, 2024 22:15
@dav1do dav1do temporarily deployed to github-tests-2024 July 26, 2024 22:26 — with GitHub Actions Inactive
Copy link
Collaborator

@smrz2001 smrz2001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@dav1do dav1do added this pull request to the merge queue Jul 28, 2024
Merged via the queue into main with commit d97c226 Jul 29, 2024
6 checks passed
@dav1do dav1do deleted the fix/aes-247-dns-errors branch July 29, 2024 00:05
@smrz2001 smrz2001 mentioned this pull request Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants