From 306d8494d6de07a611a41e4a49d61c618d44b8d0 Mon Sep 17 00:00:00 2001
From: Quentin McGaw <quentin.mcgaw@gmail.com>
Date: Tue, 19 Apr 2022 11:52:05 +0000
Subject: [PATCH] hotfix(servers): assume UDP+TCP if not precised

---
 internal/models/server.go | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/internal/models/server.go b/internal/models/server.go
index 03515fe5..d19cb924 100644
--- a/internal/models/server.go
+++ b/internal/models/server.go
@@ -32,11 +32,17 @@ type Server struct {
 func (s *Server) setDefaults() {
 	// TODO v4 precise these in servers.json rather than here
 	if s.VPN == "" {
+		// If the VPN protocol isn't specified, assume it is OpenVPN.
 		s.VPN = vpn.OpenVPN
 	}
 
-	if s.VPN == vpn.Wireguard {
+	if !s.UDP && !s.TCP {
+		// If UDP and TCP are not precised:
+		// For OpenVPN, assume TCP and UDP are supported
+		// For Wireguard, UDP must be supported.
 		s.UDP = true
-		s.TCP = false
+		if s.VPN == vpn.OpenVPN {
+			s.TCP = true
+		}
 	}
 }