[bug] reset traffic when stopping vpn

also, refactor a bit VPNManagement so that the `connect_to_management`
method does not receive connection details.

I plan to refactor management so that it is a separate object from the
control object (but we need to stablish better its relation with the
status parsing class).

- Resolves: #8834

1 files changed, 2 insertions, 4 deletions

diff --git a/src/leap/bitmask/vpn/manager.py b/src/leap/bitmask/vpn/manager.py
index 1c07976f..2b113a75 100644
--- a/src/leap/bitmask/vpn/manager.py
+++ b/src/leap/bitmask/vpn/manager.py
@@ -27,16 +27,14 @@ from ._config import _TempVPNConfig, _TempProviderConfig
 from .constants import IS_WIN
 
 
-# TODO this is very badly named. There is another class that is called
-# manager. This
+# TODO this is very badly named. There is another class that is called manager.
+# TODO Call it Tunnel? Tunnel = vpn + firewall
 
 class TunnelManager(object):
 
     def __init__(self, provider, remotes, cert_path, key_path, ca_path,
                  extra_flags):
         """
-        Initialize the VPNManager object.
-
         :param remotes: a list of gateways tuple (ip, port) looking like this:
             ((ip1, portA), (ip2, portB), ...)
         :type remotes: tuple of tuple(str, int)