diff options
| author | Simon Fondrie-Teitler <simonft@riseup.net> | 2017-05-01 22:03:42 -0400 |
|---|---|---|
| committer | Kali Kaneko (leap communications) <kali@leap.se> | 2017-05-04 21:25:51 +0200 |
| commit | 73fe34135dcfee98ecc1ee2287ea063028d550f6 (patch) | |
| tree | f89b0dc10076c842318dc5a1542c0d8142e7918d /src/leap/bitmask/bonafide | |
| parent | 004f694569a519b160677097a691c609268dbdd0 (diff) | |
[bug] correctly handle authenticate's autoconf parameter
This fixes two bugs with handling the autoconf parameter:
- It looks for "True" instead of "true" in the dispatching code to
account for json.dumps() converting true into a boolean and str()
returning the python-style capitalized version "True".
- It moves the initial definitions of offirst_bootstrap,
ongoing_bootstrap, and stuck_bootstrap into the class instantiation
method so they don't get shared between instances of the
class. Previously, this caused one instance being bootstrapped to
causes other instances to think they were also actively being
bootstrapped.
Resolves #8843
Diffstat (limited to 'src/leap/bitmask/bonafide')
| -rw-r--r-- | src/leap/bitmask/bonafide/config.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/leap/bitmask/bonafide/config.py b/src/leap/bitmask/bonafide/config.py index b3c35ba..82442c8 100644 --- a/src/leap/bitmask/bonafide/config.py +++ b/src/leap/bitmask/bonafide/config.py @@ -163,10 +163,6 @@ class Provider(object): 'openvpn': ['eip'], 'mx': ['soledad', 'smtp']} - first_bootstrap = defaultdict(None) - ongoing_bootstrap = defaultdict(None) - stuck_bootstrap = defaultdict(None) - log = Logger() def __init__(self, domain, autoconf=False, basedir=None, @@ -179,6 +175,10 @@ class Provider(object): self._disco = Discovery('https://%s' % domain) self._provider_config = None + self.first_bootstrap = defaultdict(None) + self.ongoing_bootstrap = defaultdict(None) + self.stuck_bootstrap = defaultdict(None) + is_configured = self.is_configured() if not is_configured: check_certificate = False |