search

l-n-s / wireguard-install

WireGuard VPN server installer
MIT License
656 stars 166 forks source link

DNS-Server #4

Closed Preisschild closed 5 years ago

Preisschild commented 5 years ago

The client didn't had an dns server set. IMO the script should ask on first setup which dns server to use and put it in DNS = at the client configs.

vjanssens commented 5 years ago

Thanks to this issue, I knew I had to set DNS in the client config to actually get it working. I don't know about best practices, but this was what I ended up with in /root/client-wg0.conf:

[Interface]
< some more settings >
DNS = 1.1.1.1
Preisschild commented 5 years ago

Yea, I've updated the script to automatically do this, but I hadn't the time yet to make it so that it asks the user the first time to set a dns server

l-n-s commented 5 years ago

Added a basic DNS server dialog in 2f42cf5

mgdudas commented 5 years ago

While it does appear to write the DNS line properly in the configuration file, the QR code it is generating is not pulling in the set DNS servers in the iOS Wireguard client. Any idea why? It could be an iOS app problem. I did try adding a space to see if that was it, and also decoded the qrcode to make sure it had the data in it, which it does. So not sure what the issue is.

mgdudas commented 5 years ago

Yeah, I'm going to chock this up to an iOS issue. I've reached out to wireguard to see if we can figure it out.

Preisschild commented 5 years ago

Hmm strange. I've tested it in Android and it seems to be working.

EricKuck commented 5 years ago

The iOS QR code implementation is working fine. The issue is with this script, which places DNS in the peer section rather than the interface section.

l-n-s commented 5 years ago

Right, thanks for pointing that out. Now it is fixed.