Merge branch 'develop' into 518-some-service-class-method-documentation-is-not-showing-in-the-manual-pages

Author: longshuicy

CHANGELOG.md

@@ -11,6 +11,9 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 ### Fixed
 - Fixed Sphinx autodoc skipping class methods with custom decorators. [#518](https://github.com/IN-CORE/pyincore/issues/518)
 
+### Changed
+- Remove unused insecure IN-CORE client [#581](https://github.com/IN-CORE/pyincore/issues/581)
+
 ### Added
 - Apply Black formatter [#589](https://github.com/IN-CORE/pyincore/issues/589)
 

docs/source/modules.rst

@@ -575,8 +575,6 @@ client
     :members:
 ..  autoclass:: client.IncoreClient
     :members:
-..  autoclass:: client.InsecureIncoreClient
-    :members:
 
 dataservice
 ===========

pyincore/__init__.py

@@ -8,7 +8,6 @@
 # The order of import matters. You need to import module by order of dependency
 from pyincore.client import Client
 from pyincore.client import IncoreClient
-from pyincore.client import InsecureIncoreClient
 from pyincore.hazardservice import HazardService
 from pyincore.utils.expressioneval import Parser
 from pyincore.utils.cge_ml_file_util import CGEMLFileUtil

pyincore/client.py

@@ -451,26 +451,3 @@ def clear_cache(self):
         # clear entry from service.json
         update_hash_entry("edit", hashed_url=self.hashed_service_url)
         return
-
-
-class InsecureIncoreClient(Client):
-    """IN-CORE service client class that bypasses Ambassador auth. It contains token and service root url.
-
-    Args:
-        service_url (str): Service url.
-        username (str): Username string.
-
-    """
-
-    def __init__(self, service_url: str = None, username: str = None):
-        super().__init__()
-        if service_url is None or len(service_url.strip()) == 0:
-            service_url = pyglobals.INCORE_API_PROD_URL
-        self.service_url = service_url
-        if username is None or len(username.strip()) == 0:
-            self.session.headers[
-                "x-auth-userinfo"
-            ] = pyglobals.INCORE_LDAP_TEST_USER_INFO
-        else:
-            user_info = '{"preferred_username": "' + username + '"}'
-            self.session.headers["x-auth-userinfo"] = user_info