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 wg0 DNS comment #212

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fix wg0 DNS comment #212

wants to merge 1 commit into from

Conversation

Jorundur
Copy link
Contributor

@Jorundur Jorundur commented Jun 24, 2024

Pull request

Purpose
Hope I'm not mistaken, but should we not make sure to remove the DNS line in our wg0-fix.conf file, regardless of its value? The container then sets the DNS values, right?

@bokkoman
Copy link
Contributor

I'm not sure, but i believe the environment variables overrule the wg0 dns setting.
If we know this for sure, we can ignore it in the conf file.

@bakerboy448
Copy link
Contributor

bakerboy448 commented Jun 25, 2024

With VPN_CONF you can set the name used for your WireGuard config. This is an example of how your wg0.conf file should look like. If there's a lot of extra stuff, remove it unless you know what it's there for. The WireGuard config is automatically modified to use AllowedIPs = 0.0.0.0/1,128.0.0.0/1 for compatibility with Synology/QNAP/Asustor/WSL2 systems if you append -fix, so VPN_CONF=wg0-fix. WSL2 users can also compile their own kernel if they don't wanna use this workaround.

[Interface]
PrivateKey = supersecretprivatekey
Address = xx.xx.xxx.xxx/32 # Yes, /32
DNS = x.x.x.x

[Peer]
PublicKey = publickey
AllowedIPs = 0.0.0.0/0
Endpoint = xxx.x.xxx.x:51820

@bokkoman
Copy link
Contributor

@bakerboy448 but regardless of that, does the DNS settings in the template for qbit override the DNS setting in the conf?

@bakerboy448
Copy link
Contributor

@bakerboy448 but regardless of that, does the DNS settings in the template for qbit override the DNS setting in the conf?

This is answered in hotio's documentation.

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.

3 participants