[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

1 files changed, 4 insertions, 4 deletions

diff --git a/src/leap/bitmask/bonafide/config.py b/src/leap/bitmask/bonafide/config.py
index b3c35ba1..82442c86 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