summaryrefslogtreecommitdiff
path: root/src/leap/bitmask/vpn
diff options
context:
space:
mode:
Diffstat (limited to 'src/leap/bitmask/vpn')
-rw-r--r--src/leap/bitmask/vpn/autostart.py2
-rw-r--r--src/leap/bitmask/vpn/service.py10
2 files changed, 9 insertions, 3 deletions
diff --git a/src/leap/bitmask/vpn/autostart.py b/src/leap/bitmask/vpn/autostart.py
index 858ea4ab..0572e9d5 100644
--- a/src/leap/bitmask/vpn/autostart.py
+++ b/src/leap/bitmask/vpn/autostart.py
@@ -20,7 +20,7 @@ Terminal=false
Leave an autostart file in the user's autostart path.
The bundle could in principle find its own path and add
- the path to the bitmaskd binary in the Exec entry.
+ the path to the bitmaskd binary in the Exec entry.
But for now it's simpler to do autostart only for the debian packages
or any other method that puts bitmask in the path.
On the other hand, we want to reduce the modifications that the bundle
diff --git a/src/leap/bitmask/vpn/service.py b/src/leap/bitmask/vpn/service.py
index 6d273e23..9faedd95 100644
--- a/src/leap/bitmask/vpn/service.py
+++ b/src/leap/bitmask/vpn/service.py
@@ -315,7 +315,13 @@ class VPNService(HookableService):
yield bonafide.do_provider_create(provider_id)
provider = yield bonafide.do_provider_read(provider_id)
- config = yield bonafide.do_provider_read(provider_id, 'eip')
+ try:
+ config = yield bonafide.do_provider_read(provider_id, 'eip')
+ except ValueError:
+ exc = Exception('Cannot find EIP section for provider %s. '
+ 'Is it fully bootstrapped?' % provider_id)
+ exc.expected = True
+ raise exc
sorted_gateways = self._get_gateways(config)
extra_flags = config.openvpn_configuration
@@ -324,7 +330,7 @@ class VPNService(HookableService):
anonvpn = self._has_anonvpn(provider)
ready = self.do_check(provider_id).get('vpn_ready', False)
- if not ready:
+ if not ready and anonvpn:
yield self._maybe_get_anon_cert(anonvpn, provider_id)
if not os.path.isfile(ca_path):