Move config loading into config.py and allow overriding with environment variables

Author: lukegb

hackyplayer/app.py

@@ -15,16 +15,6 @@
 app = tasks.flask_app
 app_cel = tasks.celery_app
 
-# Load default settings
-app.config.update(config.DEFAULT_CONFIG)
-
-# Load local settings
-try:
-    import config_local
-    app.config.update(config_local.CONFIG)
-except (ImportError, AttributeError):
-    pass
-
 app.config["api_route"] = "/api/v1"
 
 def get_files(target, ext_filter = []):

hackyplayer/config.py

@@ -28,7 +28,12 @@
         "NAME": "input",
         "FULLPATH": pathlib.Path("static/video/input"),
         "OUTPUT_DIR": pathlib.Path("static/video/source")
-    }]
+    }],
+    "CELERY": {
+        "broker_url": "redis://localhost",
+        "result_backend": "redis://localhost",
+        "task_ignore_result": False,
+    },
 }
 
 def celery_init_app(app):
@@ -45,13 +50,17 @@ def __call__(self, *args: object, **kwargs: object) -> object:
 
 def create_app():
     app = flask.Flask(__name__)
-    app.config.from_mapping(
-        CELERY=dict(
-            broker_url="redis://localhost",
-            result_backend="redis://localhost",
-            task_ignore_result=False,
-        ),
-    )
+
+    # Load default settings
+    app.config.update(DEFAULT_CONFIG)
+
+    # Load local settings
+    try:
+        import config_local
+        app.config.update(config_local.CONFIG)
+    except (ImportError, AttributeError):
+        pass
+
     app.config.from_prefixed_env()
     celery_init_app(app)
     return app