From 51d8fba9162b9e148a35ced69e7e035438c8b0f1 Mon Sep 17 00:00:00 2001
From: sct <sctsnipe@gmail.com>
Date: Mon, 18 Jan 2021 01:25:44 +0000
Subject: [PATCH] fix(server): support absolute paths for CONFIG_DIRECTORY

---
 ormconfig.js           | 8 ++++++--
 server/lib/settings.ts | 8 +++-----
 server/logger.ts       | 8 +++-----
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/ormconfig.js b/ormconfig.js
index 5ff4d06b..070e0598 100644
--- a/ormconfig.js
+++ b/ormconfig.js
@@ -1,6 +1,8 @@
 const devConfig = {
   type: 'sqlite',
-  database: `${process.env.CONFIG_DIRECTORY || 'config'}/db/db.sqlite3`,
+  database: process.env.CONFIG_DIRECTORY
+    ? `${process.env.CONFIG_DIRECTORY}/db/db.sqlite3`
+    : 'config/db/db.sqlite3',
   synchronize: true,
   migrationsRun: false,
   logging: false,
@@ -15,7 +17,9 @@ const devConfig = {
 
 const prodConfig = {
   type: 'sqlite',
-  database: `${process.env.CONFIG_DIRECTORY || 'config'}/db/db.sqlite3`,
+  database: process.env.CONFIG_DIRECTORY
+    ? `${process.env.CONFIG_DIRECTORY}/db/db.sqlite3`
+    : 'config/db/db.sqlite3',
   synchronize: false,
   logging: false,
   entities: ['dist/entity/**/*.js'],
diff --git a/server/lib/settings.ts b/server/lib/settings.ts
index 01e73b89..b9ad92a9 100644
--- a/server/lib/settings.ts
+++ b/server/lib/settings.ts
@@ -137,11 +137,9 @@ interface AllSettings {
   notifications: NotificationSettings;
 }
 
-const SETTINGS_PATH = path.join(
-  __dirname,
-  '../../',
-  `${process.env.CONFIG_DIRECTORY || '/config'}/settings.json`
-);
+const SETTINGS_PATH = process.env.CONFIG_DIRECTORY
+  ? `${process.env.CONFIG_DIRECTORY}/settings.json`
+  : path.join(__dirname, '../../config/settings.json');
 
 class Settings {
   private data: AllSettings;
diff --git a/server/logger.ts b/server/logger.ts
index 014bd336..824de630 100644
--- a/server/logger.ts
+++ b/server/logger.ts
@@ -42,11 +42,9 @@ const logger = winston.createLogger({
       ),
     }),
     new winston.transports.DailyRotateFile({
-      filename: path.join(
-        __dirname,
-        '../',
-        `${process.env.CONFIG_DIRECTORY || '/config'}/logs/overseerr-%DATE%.log`
-      ),
+      filename: process.env.CONFIG_DIRECTORY
+        ? `${process.env.CONFIG_DIRECTORY}/logs/overseerr-%DATE%.log`
+        : path.join(__dirname, '../config/logs/overseerr-%DATE%.log'),
       datePattern: 'YYYY-MM-DD',
       zippedArchive: true,
       maxSize: '20m',