summaryrefslogtreecommitdiff
path: root/src/leap/base
diff options
context:
space:
mode:
authorantialias <antialias@leap.se>2012-09-28 18:16:47 -0400
committerantialias <antialias@leap.se>2012-09-28 18:16:47 -0400
commit58344bb28c1c0f25ed37624ff487cc8f24821d52 (patch)
tree873a8225862e20124ee69af7c509772dab952674 /src/leap/base
parent202345940e12d4633c5d46d5ed21c3a433573d48 (diff)
Functionality to shutdown network checker when openvpn is stopped.
But thread not being successfully killed.
Diffstat (limited to 'src/leap/base')
-rw-r--r--src/leap/base/network.py40
1 files changed, 29 insertions, 11 deletions
diff --git a/src/leap/base/network.py b/src/leap/base/network.py
index 58f903e1..a1e7c880 100644
--- a/src/leap/base/network.py
+++ b/src/leap/base/network.py
@@ -1,12 +1,18 @@
# -*- coding: utf-8 -*-
from __future__ import (print_function)
+import logging
+import threading
from leap.base.checks import LeapNetworkChecker
from leap.base.constants import ROUTE_CHECK_INTERVAL
-from leap.util.coroutines import (launch_thread, process_events)
+from leap.base.exceptions import TunnelNotDefaultRouteError
+from leap.util.coroutines import (launch_thread_no_daemon, process_events)
from time import sleep
+logger = logging.getLogger(name=__name__)
+
+
class NetworkChecker(object):
"""
Manages network checking thread that makes sure we have a working network
@@ -15,12 +21,17 @@ class NetworkChecker(object):
def __init__(self, *args, **kwargs):
self.status_signals = kwargs.pop('status_signals', None)
self.watcher_cb = kwargs.pop('status_signals', None)
+ self.excp_logger = lambda exc: logger.error("%s", exc.message)
+ self.checker = LeapNetworkChecker()
def start(self):
- self._launch_recurrent_network_checks((self.watcher_cb,))
+ self.process_handle = self._launch_recurrent_network_checks((self.excp_logger,))
def stop(self):
- raise NotImplementedError
+ #TODO: Thread still not being stopped when openvpn is stopped.
+ logger.debug("stopping network checker...")
+ self.process_handle._Thread__stop()
+ logger.debug("network checked stopped.")
def run_checks(self):
pass
@@ -31,24 +42,31 @@ class NetworkChecker(object):
#which is exception so we can try by passing a lambda with logger to
#check it works.
def _network_checks_thread(self, fail_callbacks):
- print('fail_callbacks: %s' % fail_callbacks)
- print(len(fail_callbacks))
+ #TODO: replace this with waiting for a signal from openvpn
+ while True:
+ try:
+ self.checker.check_tunnel_default_interface()
+ break
+ except TunnelNotDefaultRouteError:
+ sleep(1)
+
observer_dict = dict(((
observer, process_events(observer)) for observer in fail_callbacks))
- netchecker = LeapNetworkChecker()
while True:
try:
- netchecker.check_internet_connection()
+ self.checker.check_tunnel_default_interface()
+ self.checker.check_internet_connection()
sleep(ROUTE_CHECK_INTERVAL)
except Exception as exc:
for obs in observer_dict:
observer_dict[obs].send(exc)
+ sleep(ROUTE_CHECK_INTERVAL)
- def _launch_recurrent_network_checks(fail_callbacks):
- print(type(fail_callbacks))
- watcher = launch_thread(
- network_checks_thread,
+ def _launch_recurrent_network_checks(self, fail_callbacks):
+ #we need to wrap the fail callback in a turple
+ watcher = launch_thread_no_daemon(
+ self._network_checks_thread,
(fail_callbacks,))
return watcher