Skip to content

fix(rust/client/windows): set our DNS resolvers on our interface#6931

Merged
ReactorScram merged 12 commits intomainfrom
fix/windows-set-dns-on-interface
Oct 7, 2024
Merged

fix(rust/client/windows): set our DNS resolvers on our interface#6931
ReactorScram merged 12 commits intomainfrom
fix/windows-set-dns-on-interface

Conversation

@ReactorScram
Copy link
Copy Markdown
Contributor

@ReactorScram ReactorScram commented Oct 3, 2024

Closes #6777

@ReactorScram ReactorScram self-assigned this Oct 3, 2024
@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
firezone ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 7, 2024 2:52pm

@ReactorScram ReactorScram changed the base branch from main to chore/headless-deactivate-dns-sooner October 3, 2024 22:14
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed unit tests here that only covered clap's behavior and not our code

Base automatically changed from chore/headless-deactivate-dns-sooner to main October 4, 2024 14:19
@ReactorScram
Copy link
Copy Markdown
Contributor Author

I couldn't get the IPv6 part to work on my dev system, maybe because I don't have IPv6 on this network.

It works in CI and it should be harmless even if IPv6 doesn't work. I can test IPv6 when I'm back home Monday.

@ReactorScram ReactorScram marked this pull request as ready for review October 4, 2024 16:36
Copy link
Copy Markdown
Member

@thomaseizinger thomaseizinger left a comment

Choose a reason for hiding this comment

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

Nice work!

Comment thread rust/headless-client/src/dns_control/windows.rs Outdated
Comment thread rust/headless-client/src/dns_control/windows.rs Outdated
Comment thread rust/headless-client/src/dns_control/windows.rs
Comment thread rust/headless-client/src/dns_control/windows.rs Outdated
@ReactorScram ReactorScram added this pull request to the merge queue Oct 7, 2024
Merged via the queue into main with commit 9b93fc2 Oct 7, 2024
@ReactorScram ReactorScram deleted the fix/windows-set-dns-on-interface branch October 7, 2024 15:16
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.

Issues connecting to resources from within WSL

2 participants