summaryrefslogtreecommitdiff
path: root/src/leap/base/checks.py
diff options
context:
space:
mode:
authorkali <kali@leap.se>2012-10-05 21:21:22 +0900
committerkali <kali@leap.se>2012-10-05 21:21:22 +0900
commit1cbf954d9eda71cabfa58811c09bc63cfe9465d5 (patch)
treeef7b869fb7f59c9f8cbeb328427a8f9ac854aa7c /src/leap/base/checks.py
parent31c0afa5eb9bc7566ca39099520e8adc7b531e22 (diff)
add comments to netchecks
Diffstat (limited to 'src/leap/base/checks.py')
-rw-r--r--src/leap/base/checks.py40
1 files changed, 37 insertions, 3 deletions
diff --git a/src/leap/base/checks.py b/src/leap/base/checks.py
index d02c9a3d..0dbb2846 100644
--- a/src/leap/base/checks.py
+++ b/src/leap/base/checks.py
@@ -17,8 +17,15 @@ class LeapNetworkChecker(object):
"""
all network related checks
"""
- # TODO eventually, use a more portable solution
- # like psutil
+ # TODO refactor to use psutil ---
+
+ # #718
+ # XXX get provider gateway as a parameter
+ # for constructor.
+ # def __init__(self, *args, **kwargs):
+ # ...
+ # provider_gw = kwargs.pop('provider_gw', None)
+ # self.provider_gateway = provider_gw
def run_all(self, checker=None):
if not checker:
@@ -29,12 +36,23 @@ class LeapNetworkChecker(object):
checker.check_tunnel_default_interface()
checker.check_internet_connection()
checker.is_internet_up()
+
+ # XXX We are pinging the default gateway for our connection right?
+ # kali: 2012-10-05 20:59 -- I think we should get
+ # also the default gateway and ping it instead.
checker.ping_gateway()
+ # something like: ?
+ # see __init__ above
+ # if self.provider_gateway:
+ # checker.ping_gateway(self.provider_gateway)
+
def check_internet_connection(self):
try:
# XXX remove this hardcoded random ip
+ # ping leap.se or eip provider instead...?
requests.get('http://216.172.161.165')
+
except (requests.HTTPError, requests.RequestException) as e:
raise exceptions.NoInternetConnection(e.message)
except requests.ConnectionError as e:
@@ -44,6 +62,7 @@ class LeapNetworkChecker(object):
error = "No valid internet connection found."
else:
error = "Provider server appears to be down."
+ logger.error(error)
raise exceptions.NoInternetConnection(error)
logger.debug('Network appears to be up.')
@@ -78,6 +97,7 @@ class LeapNetworkChecker(object):
if not platform.system() == "Linux":
raise NotImplementedError
+ # XXX use psutil
f = open("/proc/net/route")
route_table = f.readlines()
f.close()
@@ -102,7 +122,21 @@ class LeapNetworkChecker(object):
return default_iface, gateway
def ping_gateway(self, gateway):
- #TODO: Discuss how much packet loss (%) is acceptable.
+ # TODO: Discuss how much packet loss (%) is acceptable.
+
+ # XXX -- validate gateway
+ # -- is it a valid ip? (there's something in util)
+ # -- is it a domain?
+ # -- can we resolve? -- raise NoDNSError if not.
packet_loss = ping.quiet_ping(gateway)[0]
if packet_loss > constants.MAX_ICMP_PACKET_LOSS:
raise exceptions.NoConnectionToGateway
+
+ # XXX check for name resolution servers
+ # dunno what's the best way to do this...
+ # check for etc/resolv entries or similar?
+ # just try to resolve?
+ # is there something in psutil?
+
+ # def check_name_resolution(self):
+ # pass