[feature] hard restarts upon ping-restart received

1 files changed, 16 insertions, 19 deletions

diff --git a/src/leap/bitmask/vpn/manager.py b/src/leap/bitmask/vpn/manager.py
index 8781603d..abadc00e 100644
--- a/src/leap/bitmask/vpn/manager.py
+++ b/src/leap/bitmask/vpn/manager.py
@@ -42,36 +42,33 @@ class VPNManager(object):
         :type remotes: tuple of tuple(str, int)
         """
         # TODO we can set all the needed ports, gateways and paths in here
+        # TODO need gateways here
+        # sorting them doesn't belong in here
+        # gateways = ((ip1, portA), (ip2, portB), ...)
+
         ports = []
 
-        # this seems to be obsolete, needed to get gateways
+        # TODO fix hardcoding
         domain = "demo.bitmask.net"
+
         self._remotes = remotes
 
         self._eipconfig = _TempEIPConfig(extra_flags, cert_path, ports)
         self._providerconfig = _TempProviderConfig(domain, ca_path)
-        self._vpn = VPNControl(remotes=remotes)
 
+        host, port = self._get_management_location()
+        self._vpn = VPNControl(remotes=remotes,
+                               eipconfig=self._eipconfig,
+                               providerconfig=self._providerconfig,
+                               socket_host=host, socket_port=port)
 
     def start(self):
         """
         Start the VPN process.
-
-        VPN needs:
-        * paths for: cert, key, ca
-        * gateway, port
-        * domain name
         """
-        host, port = self._get_management_location()
-
-        # TODO need gateways here
-        # sorting them doesn't belong in here
-        # gateways = ((ip1, portA), (ip2, portB), ...)
-
-        self._vpn.start(eipconfig=self._eipconfig,
-                        providerconfig=self._providerconfig,
-                        socket_host=host, socket_port=port)
-        return True
+        result = self._vpn.start()
+        print "RESULT START --->", result
+        return result
 
     def stop(self):
         """
@@ -81,8 +78,8 @@ class VPNManager(object):
         :rtype: bool
         """
         # TODO how to return False if this fails
-        self._vpn.stop(False, False)  # TODO review params
-        return True
+        result = self._vpn.stop(False, False)  # TODO review params
+        return result
 
 
     @property