summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorjwijenbergh <jeroenwijenbergh@protonmail.com>2022-11-25 09:32:32 +0100
committerjwijenbergh <jeroenwijenbergh@protonmail.com>2022-11-25 09:32:32 +0100
commitb4ff890ec2b459148d893499a34a6d2954530369 (patch)
tree8d679e1502e4e35c2780f31f1b137613fccd2295
parent106636991f6f9eb97982d1563b696964ebeaaa40 (diff)
Discovery + Exports: Rename variable to not clash with server pkg
-rw-r--r--exports/servers.go26
-rw-r--r--internal/discovery/discovery.go22
2 files changed, 24 insertions, 24 deletions
diff --git a/exports/servers.go b/exports/servers.go
index b785201..7536465 100644
--- a/exports/servers.go
+++ b/exports/servers.go
@@ -165,7 +165,7 @@ func freeCListStrings(allStrings **C.char, totalStrings C.size_t) {
// It gets the base information for a server as well
func getCPtrServer(state *client.Client, base *client.ServerBase) *C.server {
// Allocation using malloc and the size of the struct
- server := (*C.server)(C.malloc(C.size_t(unsafe.Sizeof(C.server{}))))
+ cServer := (*C.server)(C.malloc(C.size_t(unsafe.Sizeof(C.server{}))))
// String allocation and translate the display name
identifier := base.URL
countryCode := ""
@@ -177,28 +177,28 @@ func getCPtrServer(state *client.Client, base *client.ServerBase) *C.server {
locations = state.Discovery.GetSecureLocationList()
}
- server.identifier = C.CString(identifier)
- server.display_name = C.CString(state.GetTranslated(base.DisplayName))
- server.country_code = C.CString(countryCode)
- server.server_type = C.CString(base.Type)
+ cServer.identifier = C.CString(identifier)
+ cServer.display_name = C.CString(state.GetTranslated(base.DisplayName))
+ cServer.country_code = C.CString(countryCode)
+ cServer.server_type = C.CString(base.Type)
// Call the helper to get the list of support contacts
- server.total_support_contact, server.support_contact = getCPtrListStrings(
+ cServer.total_support_contact, cServer.support_contact = getCPtrListStrings(
base.SupportContact,
)
locationsStruct := (*C.serverLocations)(C.malloc(C.size_t(unsafe.Sizeof(C.servers{}))))
locationsStruct.total_locations, locationsStruct.locations = getCPtrListStrings(locations)
- server.locations = locationsStruct
+ cServer.locations = locationsStruct
profiles := base.GetValidProfiles(state.SupportsWireguard)
- server.profiles = getCPtrProfiles(&profiles)
+ cServer.profiles = getCPtrProfiles(&profiles)
// No endtime is given if we get servers when it has been partially initialised
if base.EndTime.IsZero() {
- server.expire_time = C.ulonglong(0)
+ cServer.expire_time = C.ulonglong(0)
} else {
// The expire time should be stored as an unsigned long long in unix itme
- server.expire_time = C.ulonglong(base.EndTime.Unix())
+ cServer.expire_time = C.ulonglong(base.EndTime.Unix())
}
- return server
+ return cServer
}
// Function for freeing a single server
@@ -231,8 +231,8 @@ func getCPtrServers(
serversPtr := (**C.server)(C.malloc(totalServers * C.size_t(unsafe.Sizeof(uintptr(0)))))
servers := (*[1<<30 - 1]*C.server)(unsafe.Pointer(serversPtr))[:totalServers:totalServers]
index := 0
- for _, server := range serverMap {
- cServer := getCPtrServer(state, &server.Base)
+ for _, currentServer := range serverMap {
+ cServer := getCPtrServer(state, &currentServer.Base)
servers[index] = cServer
index += 1
}
diff --git a/internal/discovery/discovery.go b/internal/discovery/discovery.go
index b0b20fa..9102593 100644
--- a/internal/discovery/discovery.go
+++ b/internal/discovery/discovery.go
@@ -73,9 +73,9 @@ func (discovery *Discovery) DetermineOrganizationsUpdate() bool {
func (discovery *Discovery) GetSecureLocationList() []string {
var locations []string
- for _, server := range discovery.Servers.List {
- if server.Type == "secure_internet" {
- locations = append(locations, server.CountryCode)
+ for _, currentServer := range discovery.Servers.List {
+ if currentServer.Type == "secure_internet" {
+ locations = append(locations, currentServer.CountryCode)
}
}
return locations
@@ -85,9 +85,9 @@ func (discovery *Discovery) GetServerByURL(
url string,
_type string,
) (*types.DiscoveryServer, error) {
- for _, server := range discovery.Servers.List {
- if server.BaseURL == url && server.Type == _type {
- return &server, nil
+ for _, currentServer := range discovery.Servers.List {
+ if currentServer.BaseURL == url && currentServer.Type == _type {
+ return &currentServer, nil
}
}
return nil, types.NewWrappedError(
@@ -100,9 +100,9 @@ func (discovery *Discovery) GetServerByCountryCode(
code string,
_type string,
) (*types.DiscoveryServer, error) {
- for _, server := range discovery.Servers.List {
- if server.CountryCode == code && server.Type == _type {
- return &server, nil
+ for _, currentServer := range discovery.Servers.List {
+ if currentServer.CountryCode == code && currentServer.Type == _type {
+ return &currentServer, nil
}
}
return nil, types.NewWrappedError(
@@ -136,12 +136,12 @@ func (discovery *Discovery) GetSecureHomeArgs(
// Get a server with the base url
url := org.SecureInternetHome
- server, serverErr := discovery.GetServerByURL(url, "secure_internet")
+ currentServer, serverErr := discovery.GetServerByURL(url, "secure_internet")
if serverErr != nil {
return nil, nil, types.NewWrappedError(errorMessage, serverErr)
}
- return org, server, nil
+ return org, currentServer, nil
}
// https://github.com/eduvpn/documentation/blob/v3/SERVER_DISCOVERY.md