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

gui/backend.go

@@ -30,9 +30,9 @@ func SwitchOff() {
 	backend.SwitchOff()
 }
 
-//export UseGateway
-func UseGateway(label string) {
-	backend.UseGateway(label)
+//export UseLocation
+func UseLocation(label string) {
+	backend.UseLocation(label)
 }
 
 //export UseTransport

gui/handlers.cpp

@@ -42,9 +42,9 @@ void Backend::donateSeen()
     DonateSeen();
 }
 
-void Backend::useGateway(QString label)
+void Backend::useLocation(QString label)
 {
-    UseGateway(toGoStr(label));
+    UseLocation(toGoStr(label));
 }
 
 void Backend::login(QString username, QString password)

gui/handlers.h

@@ -36,7 +36,7 @@ public slots:
     void switchOff();
     void donateAccepted();
     void donateSeen();
-    void useGateway(QString username);
+    void useLocation(QString username);
     void login(QString username, QString password);
     void resetError(QString errlabel);
     void resetNotification(QString label);

gui/qml/main.qml

@@ -163,7 +163,7 @@ Window {
                     model: [qsTr("Automatic")]
                     onActivated: {
                         console.debug("Selected gateway:", currentText)
-                        backend.useGateway(currentText.toString())
+                        backend.useLocation(currentText.toString())
                     }
                 }
             } // end column

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()
 }

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)

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

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

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