Continued:

- local variables changed to local array

diff --git a/fba/models/error_log.py b/fba/models/error_log.py
index 14d910158086223034d57356645276d7ea04a500..4ad8f977a422450fab0e3903f4ae7aa4f7eb9ebc 100644 (file)
--- a/fba/models/error_log.py
+++ b/fba/models/error_log.py
@@ -29,8 +29,10 @@ logger = logging.getLogger(__name__)
 #logger.setLevel(logging.DEBUG)
 
 # "Cached" configuration values
-_write_error_log   = config.get("write_error_log")
-_error_log_cleanup = config.get("error_log_cleanup")
+_config = {
+    "write_error_log"  : config.get("write_error_log")
+    "error_log_cleanup": config.get("error_log_cleanup")
+}
 
 def add(domain: str, error: dict) -> None:
     logger.debug("domain='%s',error[]='%s' - CALLED!", domain, type(error))
@@ -38,7 +40,7 @@ def add(domain: str, error: dict) -> None:
 
     if blacklist.is_blacklisted(domain):
         raise RuntimeError(f"domain='{domain}' is blacklisted but function was invoked")
-    elif not _write_error_log:
+    elif not _config["write_error_log"]:
         raise RuntimeError("Logging of errors is disabled but function was invoked")
 
     logger.debug("error[]='%s' - BEFORE!", type(error))
@@ -61,7 +63,8 @@ def add(domain: str, error: dict) -> None:
         ])
 
     # Cleanup old entries
-    logger.debug("Purging old records (distance: %d)", _error_log_cleanup)
-    database.cursor.execute("DELETE FROM error_log WHERE created < ?", [time.time() - _error_log_cleanup])
+    logger.debug("Purging old records (distance: %d)", _config["error_log_cleanup"])
+    database.cursor.execute("DELETE FROM error_log WHERE created < ?", [time.time() - _config["error_log_cleanup"]
+    ])
 
     logger.debug("EXIT!")