diff options
author | Kali Kaneko (leap communications) <kali@leap.se> | 2017-04-18 22:30:41 +0200 |
---|---|---|
committer | Kali Kaneko (leap communications) <kali@leap.se> | 2017-04-19 20:14:29 +0200 |
commit | 6756e922e9fe5e8f678d3ccaefadb27c23f400ae (patch) | |
tree | 52b5b7f685eab94dcc24baf0d8846755e6cd4edc /src/leap/bitmask/vpn | |
parent | 71c341b9aea1097e0d8f4ccc2eee9728599ea8b0 (diff) |
[bug] fail if tried to stop vpn when not running
Diffstat (limited to 'src/leap/bitmask/vpn')
-rw-r--r-- | src/leap/bitmask/vpn/privilege.py | 3 | ||||
-rw-r--r-- | src/leap/bitmask/vpn/service.py | 2 |
2 files changed, 3 insertions, 2 deletions
diff --git a/src/leap/bitmask/vpn/privilege.py b/src/leap/bitmask/vpn/privilege.py index 3c66c7a5..135acbb8 100644 --- a/src/leap/bitmask/vpn/privilege.py +++ b/src/leap/bitmask/vpn/privilege.py @@ -51,11 +51,10 @@ def uninstall_helpers(): class NoPolkitAuthAgentAvailable(Exception): message = 'No polkit authentication agent available. Please run one.' - pass class NoPkexecAvailable(Exception): - pass + message = 'Could not find pkexec in the system' def is_missing_policy_permissions(): diff --git a/src/leap/bitmask/vpn/service.py b/src/leap/bitmask/vpn/service.py index eefccad1..2766b8c0 100644 --- a/src/leap/bitmask/vpn/service.py +++ b/src/leap/bitmask/vpn/service.py @@ -88,6 +88,8 @@ class VPNService(HookableService): self._vpn.stop() self._started = False return {'result': 'stopped'} + else: + raise Exception('VPN was not running') def do_status(self): status = { |