Let's rename the API to use location instead of gateway

5 files changed, 5 insertions, 15 deletions

diff --git a/pkg/backend/actions.go b/pkg/backend/actions.go
index 6e143f9..1860b09 100644
--- a/pkg/backend/actions.go
+++ b/pkg/backend/actions.go
@@ -20,14 +20,6 @@ func stopVPN() {
 	}
 }
 
-// TODO return bool?
-func useGateway(label string) {
-	err := ctx.bm.UseGateway(label)
-	if err != nil {
-		log.Println(err)
-	}
-}
-
 func getGateway() string {
 	return ctx.bm.GetCurrentGateway()
 }
diff --git a/pkg/backend/api.go b/pkg/backend/api.go
index 761c03d..f1fed57 100644
--- a/pkg/backend/api.go
+++ b/pkg/backend/api.go
@@ -57,8 +57,8 @@ func SwitchOff() {
 
 // TODO implement Reconnect - do not tear whole fw down in between
 
-func UseGateway(label string) {
-	ctx.bm.UseGateway(string(label))
+func UseLocation(label string) {
+	ctx.bm.UseGateway(label)
 	time.Sleep(200 * time.Millisecond)
 	SwitchOff()
 	time.Sleep(500 * time.Millisecond)
diff --git a/pkg/backend/webapi.go b/pkg/backend/webapi.go
index 3e4efd3..903112e 100644
--- a/pkg/backend/webapi.go
+++ b/pkg/backend/webapi.go
@@ -51,8 +51,7 @@ func webGatewaySet(w http.ResponseWriter, r *http.Request) {
 		}
 		gwLabel := r.FormValue("gw")
 		fmt.Fprintf(w, "selected gateway: %s\n", gwLabel)
-		// FIXME catch error here, return it (error code)
-		useGateway(gwLabel)
+		ctx.bm.UseGateway(gwLabel)
 		// TODO make sure we don't tear the fw down on reconnect...
 		SwitchOff()
 		// a little sleep is needed, though, because iptables takes some time
diff --git a/pkg/bitmask/bitmask.go b/pkg/bitmask/bitmask.go
index cb7be29..eb2a833 100644
--- a/pkg/bitmask/bitmask.go
+++ b/pkg/bitmask/bitmask.go
@@ -27,7 +27,7 @@ type Bitmask interface {
 	InstallHelpers() error
 	VPNCheck() (helpers bool, priviledge bool, err error)
 	ListLocationFullness(protocol string) map[string]float64
-	UseGateway(name string) error
+	UseGateway(name string)
 	GetCurrentGateway() string
 	GetCurrentLocation() string
 	UseTransport(transport string) error
diff --git a/pkg/vpn/openvpn.go b/pkg/vpn/openvpn.go
index 1447458..e4d2781 100644
--- a/pkg/vpn/openvpn.go
+++ b/pkg/vpn/openvpn.go
@@ -235,9 +235,8 @@ func (b *Bitmask) ListLocationFullness(transport string) map[string]float64 {
 }
 
 // UseGateway selects a gateway, by label, as the default gateway
-func (b *Bitmask) UseGateway(label string) error {
+func (b *Bitmask) UseGateway(label string) {
 	b.bonafide.SetManualGateway(label)
-	return nil
 }
 
 // UseTransport selects an obfuscation transport to use