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.

Sign up for GitHub

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

Jump to bottom

feat: Make DNS server configurable #3611

Closed
wants to merge 1 commit into from
Closed

feat: Make DNS server configurable #3611

wants to merge 1 commit into from

Conversation

twpayne
Copy link
Owner

@twpayne twpayne commented Feb 29, 2024

Fixes #3605.

@ErrrorMaxx with this you should be able to set the DNS server in your config file with, for example:

[dns]
    network = "udp"
    address = "1.1.1.1:53"

Note that this only applies once chezmoi has read its config file. Specifically, when you call chezmoi init --use-builtin-git github.com/ErrrorMax/dotfiles the lookup of github.com with not use the configured DNS as chezmoi will not have read its config file yet.

Would you be able to test this?

@ErrrorMaxx
Copy link
Contributor

ErrrorMaxx commented Feb 29, 2024 •
edited
Loading

Compiled on Windows:

$ go version
go version go1.22.0 windows/amd64

$ git status
On branch fix-3605
Your branch is up to date with 'origin/fix-3605'.

Running in Termux:

$ ./chezmoi data
{
  "chezmoi": {
    "config": {
      "dns": {
        "address": "1.1.1.1:53",
        "network": "udp"
      }
...

$ ./chezmoi diff -R
chezmoi: Get "https://github.com/macvk/dnsleaktest/raw/HEAD/dnsleaktest.py": dial tcp: lookup github.com on [::1]:53: read udp [::1]:59895->[::1]:53: read: connection refused

Error didn't changed

@twpayne
Copy link
Owner Author

twpayne commented Feb 29, 2024

Thanks for testing. I guess a different solution is needed 😕

@twpayne twpayne closed this Feb 29, 2024
@twpayne twpayne deleted the fix-3605 branch February 29, 2024 22:41
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat: Native Termux builds
2 participants
@twpayne @ErrrorMaxx