Merge pull request #188 from ral-facilities/feature/configure-log-fil…

…e-location-#182 Allow Log File Location to be Configurable
ral-facilities · Nov 18, 2020 · 463e3db · 463e3db
2 parents 1596f73 + 45b5896
commit 463e3db
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/config.json.example b/config.json.example
@@ -4,6 +4,7 @@
   "ICAT_URL": "https://localhost.localdomain:8181",
   "icat_check_cert": false,
   "log_level": "WARN",
+  "log_location": "/home/user1/datagateway-api/logs.log",
   "debug_mode": false,
   "generate_swagger": false,
   "host": "127.0.0.1",

diff --git a/datagateway_api/common/config.py b/datagateway_api/common/config.py
@@ -46,6 +46,12 @@ def get_log_level(self):
         except KeyError:
             sys.exit("Missing config value, log_level")
 
+    def get_log_location(self):
+        try:
+            return self.config["log_location"]
+        except KeyError:
+            sys.exit("Missing config value, log_location")
+
     def is_debug_mode(self):
         try:
             return self.config["debug_mode"]

diff --git a/datagateway_api/common/logger_setup.py b/datagateway_api/common/logger_setup.py
@@ -3,8 +3,7 @@
 
 from datagateway_api.common.config import config
 
-log_level = "DEBUG"
-LOG_FILE_NAME = Path(__file__).parent.parent / "logs.log"
+LOG_FILE_NAME = Path(config.get_log_location())
 logger_config = {
     "version": 1,
     "formatters": {