fix default mtu handling

h44z · Dec 18, 2020 · ec60dd1 · ec60dd1
1 parent 6fd4089
commit ec60dd1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/internal/server/usermanager.go b/internal/server/usermanager.go
@@ -390,6 +390,9 @@ func (u *UserManager) validateOrCreateDevice(dev wgtypes.Device, ipAddresses []s
 		device.Mtu = 0
 		device.PersistentKeepalive = 16 // Default
 		device.IPsStr = strings.Join(ipAddresses, ", ")
+		if mtu == wireguard.WireGuardDefaultMTU {
+			mtu = 0
+		}
 		device.Mtu = mtu
 
 		res := u.db.Create(&device)

diff --git a/internal/wireguard/net.go b/internal/wireguard/net.go
@@ -7,6 +7,8 @@ import (
 	"github.com/milosgajdos/tenus"
 )
 
+const WireGuardDefaultMTU = 1420
+
 func (m *Manager) GetIPAddress() ([]string, error) {
 	wgInterface, err := tenus.NewLinkFrom(m.Cfg.DeviceName)
 	if err != nil {
@@ -107,7 +109,7 @@ func (m *Manager) SetMTU(mtu int) error {
 	}
 
 	if mtu == 0 {
-		mtu = 1420 // WireGuard default MTU
+		mtu = WireGuardDefaultMTU
 	}
 
 	if err := wgInterface.SetLinkMTU(mtu); err != nil {