admin/gluetun
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: panic for certain no server found errors

Browse Source
This commit is contained in:
Quentin McGaw (desktop)
2021-08-23 21:19:53 +00:00
parent b3d8b78205
commit 0d6800a515
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
internal/provider/cyberghost/filter_test.go
View File

@@ -19,9 +19,9 @@ func Test_Cyberghost_filterServers(t *testing.T) {
filteredServers []models.CyberghostServer filteredServers []models.CyberghostServer
err error err error
}{ }{
"no servers": { "no server": {
selection: configuration.ServerSelection{VPN: constants.OpenVPN}, selection: configuration.ServerSelection{VPN: constants.OpenVPN},
err: errors.New("no server found: for VPN openvpn; protocol udp"), err: errors.New("no server found: for VPN openvpn; protocol udp; groups Premium UDP Asia, Premium UDP Europe, Premium UDP USA"), //nolint:lll
}, },
"servers without filter defaults to UDP": { "servers without filter defaults to UDP": {
servers: []models.CyberghostServer{ servers: []models.CyberghostServer{

2
internal/provider/utils/formatting.go
View File

@@ -27,7 +27,7 @@ func NoServerFoundError(selection configuration.ServerSelection) (err error) {
} }
messageParts = append(messageParts, "protocol "+protocol) messageParts = append(messageParts, "protocol "+protocol)
switch len(selection.Countries) { switch len(selection.Groups) {
case 0: case 0:
case 1: case 1:
part := "group " + selection.Groups[0] part := "group " + selection.Groups[0]