do not store _has_auth variable

Author: JanCaha

mergin/client.py

@@ -102,7 +102,6 @@ def __init__(
         self.client_version = "Python-client/" + __version__
         if plugin_version is not None:  # this could be e.g. "Plugin/2020.1 QGIS/3.14"
             self.client_version += " " + plugin_version
-        self._has_auth = auth_token is not None or login is not None or password is not None
         self.setup_logging()
         if auth_token:
             try:
@@ -196,28 +195,24 @@ def user_agent_info(self):
     def _check_token(f):
         """Wrapper for creating/renewing authorization token."""
 
-        def wrapper(self, *args):
-
-            # if user has not provided auth information (token or login/password) it does not make sense to try these checks
-            # the client without auth can still be used to access public information like server config
-            if self._has_auth:
-                if self._auth_session:
-                    # Refresh auth token if it expired or will expire very soon
-                    delta = self._auth_session["expire"] - datetime.now(timezone.utc)
-                    if delta.total_seconds() < 5:
-                        self.log.info("Token has expired - refreshing...")
-                        if self._auth_params.get("login", None) and self._auth_params.get("password", None):
-                            self.log.info("Token has expired - refreshing...")
-                            self.login(self._auth_params["login"], self._auth_params["password"])
-                        else:
-                            raise AuthTokenExpiredError("Token has expired - please re-login")
-                else:
-                    # Create a new authorization token
-                    self.log.info(f"No token - login user: {self._auth_params.get('login', None)}")
+        def wrapper(self, *args, **kwargs):
+            if self._auth_session:
+                # Refresh auth token if it expired or will expire very soon
+                delta = self._auth_session["expire"] - datetime.now(timezone.utc)
+                if delta.total_seconds() < 5:
+                    self.log.info("Token has expired - refreshing...")
                     if self._auth_params.get("login", None) and self._auth_params.get("password", None):
+                        self.log.info("Token has expired - refreshing...")
                         self.login(self._auth_params["login"], self._auth_params["password"])
                     else:
-                        raise ClientError("Missing login or password")
+                        raise AuthTokenExpiredError("Token has expired - please re-login")
+            else:
+                # Create a new authorization token
+                self.log.info(f"No token - login user: {self._auth_params.get('login', None)}")
+                if self._auth_params.get("login", None) and self._auth_params.get("password", None):
+                    self.login(self._auth_params["login"], self._auth_params["password"])
+                else:
+                    raise ClientError("Missing login or password")
 
             return f(self, *args)