feat: add configuration option to set authenticator and its credentia…

…ls in the search API #350

BREAKING CHANGE: This commits adds a mandatory config option, so is a major change.

datagateway_api/config.json.example

@@ -12,7 +12,10 @@
     "search_api": {
         "extension": "/search-api",
         "icat_url": "https://localhost:8181",
-        "icat_check_cert": false
+        "icat_check_cert": false,
+        "plugin": "anon",
+        "username": "",
+        "password": ""
     },
     "flask_reloader": false,
     "log_level": "WARN",

datagateway_api/src/common/config.py

@@ -130,6 +130,9 @@ class SearchAPI(BaseModel):
     extension: StrictStr
     icat_check_cert: StrictBool
     icat_url: StrictStr
+    plugin: StrictStr
+    username: StrictStr
+    password: StrictStr
 
     _validate_extension = validator("extension", allow_reuse=True)(validate_extension)
 

datagateway_api/src/search_api/session_handler.py

@@ -1,6 +1,6 @@
 from functools import wraps
 import logging
-
+from datagateway_api.src.common.config import Config
 from icat.exception import ICATSessionError
 
 from datagateway_api.src.datagateway_api.icat.icat_client_pool import ICATClient
@@ -33,8 +33,7 @@ def wrapper_client_manager(*args, **kwargs):
             SessionHandler.client.getRemainingMinutes()
         except ICATSessionError as e:
             log.debug("Current client session expired: %s", e.args)
-            SessionHandler.client.login("anon", {})
-
+            SessionHandler.client.login(Config.config.search_api.plugin, {"username": Config.config.search_api.username, "password" : Config.config.search_api.password})
         return method(*args, **kwargs)
 
     return wrapper_client_manager