base: Move readconfig() call before session setup

crobinso · crobinso · commit 8ef832335090 · 2020-01-08T16:13:40.000-05:00
Otherwise we don't pull api_key and other values from
the rc file, which is needed for proper session init

Signed-off-by: Cole Robinson &lt;crobinso@redhat.com&gt;
diff --git a/bugzilla/base.py b/bugzilla/base.py
@@ -471,6 +471,10 @@ def connect(self, url=None):
             url = self.url
         url = self.fix_url(url)
 
+        self.url = url
+        # we've changed URLs - reload config
+        self.readconfig(overwrite=False)
+
         self._session = _BugzillaSession(url, self.user_agent,
                 cookiejar=self._cookiejar,
                 sslverify=self._sslverify,
@@ -480,10 +484,6 @@ def connect(self, url=None):
         backendclass = self._get_backend_class()
         self._backend = backendclass(url, self._session)
 
-        self.url = url
-        # we've changed URLs - reload config
-        self.readconfig(overwrite=False)
-
         if (self.user and self.password):
             log.info("user and password present - doing login()")
             self.login()
