From 1c418514c6f3ea57570023833ef66d2c9d3473f2 Mon Sep 17 00:00:00 2001 From: "Kali Kaneko (leap communications)" Date: Thu, 1 Jun 2017 01:33:09 +0200 Subject: [feature] log output of subprocess call --- src/leap/bitmask/vpn/fw/firewall.py | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/src/leap/bitmask/vpn/fw/firewall.py b/src/leap/bitmask/vpn/fw/firewall.py index 7f3f404..95130b2 100644 --- a/src/leap/bitmask/vpn/fw/firewall.py +++ b/src/leap/bitmask/vpn/fw/firewall.py @@ -23,9 +23,13 @@ import commands import os import subprocess +from twisted.logger import getLogger + from leap.bitmask.vpn.constants import IS_MAC from leap.common.events import catalog, emit_async +log = getLogger() + # TODO -- subclass it for osx/windows, not only for linux. @@ -82,14 +86,16 @@ class FirewallManager(object): if restart: cmd.append("restart") - # FIXME -- use a processprotocol - exitCode = subprocess.call(cmd + gateways) - emit_async(catalog.VPN_STATUS_CHANGED) - - if exitCode == 0: - return True + try: + result = subprocess.check_output( + cmd + gateways, + stderr=subprocess.STDOUT) + except Exception: + log.failure('Error launching the firewall') else: - return False + log.debug(result) + emit_async(catalog.VPN_STATUS_CHANGED) + return True def stop(self): """ -- cgit v1.2.3