Merge pull request caprover#2234 from dshook/goaccess-fixes

Goaccess fixes
next-stack · Jan 6, 2025 · 44751c2 · 44751c2
2 parents 584dbd5 + ce0911e
commit 44751c2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/routes/user/system/SystemRouter.ts b/src/routes/user/system/SystemRouter.ts
@@ -369,7 +369,10 @@ router.get('/goaccess/:appName/files', async function (req, res, next) {
         })
         .then(function (data) {
             appDefinition = data
-            return fs.readdir(directoryPath)
+            return fs.readdir(directoryPath).catch((e) => {
+                Logger.d('No goaccess logs found')
+                return []
+            })
         })
         .then(function (files) {
             return Promise.all(

diff --git a/src/user/system/CaptainManager.ts b/src/user/system/CaptainManager.ts
@@ -145,6 +145,9 @@ class CaptainManager {
             .then(function () {
                 return fs.ensureFile(CaptainConstants.baseNginxConfigPath)
             })
+            .then(function () {
+                return fs.ensureDir(CaptainConstants.nginxSharedLogsPathOnHost)
+            })
             .then(function () {
                 return fs.ensureDir(CaptainConstants.registryPathOnHost)
             })