Revize 53b0c8bc

Re #8707 - Log level mapping via dictionary, minor adjustments during review

src/config/configuration.py

@@ -18 +18 @@
 SERVER_BASE_URL = "ServerBaseURL"
 SERVER_LOG_LEVEL = "LogLevel"
 
+LOG_LEVEL_MAPPING = {
+    "DEBUG": logging.DEBUG,
+    "INFO": logging.INFO,
+    "WARNING": logging.WARNING,
+    "ERROR": logging.ERROR,
+    "CRITICAL": logging.CRITICAL
+}
+
 
 class Configuration:
     """
@@ -84 +92 @@
 
     # set log level based on config file
     app_logger = logging.getLogger(LOG_NAME)
-    if config.log_level == "DEBUG":
-        app_logger.setLevel(logging.DEBUG)
-    elif config.log_level == "INFO":
-        app_logger.setLevel(logging.INFO)
-    elif config.log_level == "WARNING":
-        app_logger.setLevel(logging.WARNING)
-    elif config.log_level == "ERROR":
-        app_logger.setLevel(logging.ERROR)
-    else:
-        app_logger.setLevel(logging.DEBUG)#
+    app_logger.setLevel(LOG_LEVEL_MAPPING.get(config.log_level, logging.DEBUG))
 
     app_logger.addHandler(handler)
 

src/controllers/certificates_controller.py

@@ -221 +221 @@
             state = self.certificate_service.get_certificate_state(v)
             data["status"] = state
         except CertificateNotFoundException:
-            Logger.error(f"No such certificate found 'ID = {id}.")
+            Logger.error(f"No such certificate found 'ID = {id}'.")
 
         return {"success": True, "data": data}, C_SUCCESS