Remove indefinite option

src/datastore/DataStore.ts

@@ -249,9 +249,10 @@ class DataStore {
         const goAccessInfo = (self.data.get(GOACCESS_INFO) ||
             {}) as GoAccessInfo
         goAccessInfo.isEnabled = goAccessInfo.isEnabled || false
-        if (!goAccessInfo.data) {
+        if (!goAccessInfo.data || !goAccessInfo.isEnabled) {
             goAccessInfo.data = {
                 rotationFrequencyCron: '0 0 1 * *', // monthly
+                logRetentionDays: 180,
             }
         }
         return Promise.resolve(goAccessInfo)

src/models/GoAccessInfo.ts

@@ -2,6 +2,6 @@ export class GoAccessInfo {
     public isEnabled: boolean
     public data: {
         rotationFrequencyCron: string
-        logRetentionDays?: number
+        logRetentionDays: number
     }
 }

src/user/system/CaptainManager.ts

@@ -745,7 +745,7 @@ class CaptainManager {
                             {
                                 key: 'LOG_RETENTION_DAYS',
                                 value: (
-                                    goAccessInfo.data.logRetentionDays ?? -1
+                                    goAccessInfo.data.logRetentionDays ?? 180
                                 ).toString(),
                             },
                         ],