FYI: Migrate LogPeriodicDeleteService to DataMaintainanceBackgroundService

Covid19Radar/Covid19Radar.Android/Covid19Radar.Android.csproj

@@ -175,7 +175,6 @@
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="Services\CloseApplicationService.cs" />
     <Compile Include="Services\DeviceCheckService.cs" />
-    <Compile Include="Services\Logs\LogPeriodicDeleteService.cs" />
     <Compile Include="Renderers\CustomDatePickerRenderer.cs" />
     <Compile Include="Services\PreferencesService.cs" />
     <Compile Include="Services\SecureStorageService.cs" />
@@ -192,6 +191,7 @@
     <Compile Include="Services\IPlatformService.cs" />
     <Compile Include="Services\LocalPathService.cs" />
     <Compile Include="Services\EventLogSubmissionBackgroundService.cs" />
+    <Compile Include="Services\DataMaintainanceBackgroundService.cs" />
   </ItemGroup>
   <ItemGroup>
     <None Include="appcenter-pre-build.sh" />
@@ -601,7 +601,6 @@
   <ItemGroup>
     <Folder Include="Renderers\" />
     <Folder Include="Resources\xml\" />
-    <Folder Include="Services\Logs\" />
     <Folder Include="Resources\drawable-zh-xxhdpi\" />
     <Folder Include="Resources\drawable-zh-xhdpi\" />
     <Folder Include="Services\Migration\" />

Covid19Radar/Covid19Radar.Android/MainApplication.cs

@@ -54,6 +54,9 @@ private Lazy<ILoggerService> _loggerService
         private Lazy<IExposureConfigurationRepository> _exposureConfigurationRepository
             = new Lazy<IExposureConfigurationRepository>(() => ContainerLocator.Current.Resolve<IExposureConfigurationRepository>());
 
+        private Lazy<AbsDataMaintainanceBackgroundService> _dataMaintainanceService
+            = new Lazy<AbsDataMaintainanceBackgroundService>(() => ContainerLocator.Current.Resolve<AbsDataMaintainanceBackgroundService>());
+
         public MainApplication(IntPtr handle, JniHandleOwnership transfer) : base(handle, transfer)
         {
         }
@@ -95,6 +98,7 @@ private void ScheduleBackgroundTasks()
             {
                 _loggerService.Value.Exception("Failed to schedule ExposureDetectionBackgroundService", exception);
             }
+
             try
             {
                 _eventLogSubmissionBackgroundService.Value.Schedule();
@@ -103,6 +107,15 @@ private void ScheduleBackgroundTasks()
             {
                 _loggerService.Value.Exception("Failed to schedule EventLogSubmissionBackgroundService", exception);
             }
+
+            try
+            {
+                _dataMaintainanceService.Value.Schedule();
+            }
+            catch (Exception exception)
+            {
+                _loggerService.Value.Exception("Failed to schedule DataMaintainanceBackgroundService", exception);
+            }
         }
 
         private void SetupENClient(ExposureNotificationClient client)
@@ -118,7 +131,7 @@ private void RegisterPlatformTypes(IContainer container)
             // Services
             container.Register<IBackupAttributeService, BackupAttributeService>(Reuse.Singleton);
             container.Register<ILocalPathService, LocalPathService>(Reuse.Singleton);
-            container.Register<ILogPeriodicDeleteService, LogPeriodicDeleteService>(Reuse.Singleton);
+            container.Register<AbsDataMaintainanceBackgroundService, DataMaintainanceBackgroundService>(Reuse.Singleton);
             container.Register<ISecureStorageDependencyService, Services.SecureStorageService>(Reuse.Singleton);
             container.Register<IPreferencesService, PreferencesService>(Reuse.Singleton);
             container.Register<IApplicationPropertyService, ApplicationPropertyService>(Reuse.Singleton);

Covid19Radar/Covid19Radar.Android/Services/DataMaintainanceBackgroundService.cs

@@ -0,0 +1,106 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
+
+using System;
+using Android.Content;
+using Android.Runtime;
+using AndroidX.Work;
+using Covid19Radar.Services;
+using Covid19Radar.Services.Logs;
+using Java.Util.Concurrent;
+using Prism.Ioc;
+using Xamarin.Essentials;
+
+namespace Covid19Radar.Droid.Services.Logs
+{
+    public class DataMaintainanceBackgroundService : AbsDataMaintainanceBackgroundService
+    {
+        private const string CURRENT_WORK_NAME = "data_maintainance_worker_20220628";
+
+        private static readonly long INTERVAL_IN_HOURS = 24;
+        private static readonly long BACKOFF_DELAY_IN_MINUTES = 60;
+
+        public DataMaintainanceBackgroundService(
+            ILogFileService logFileService,
+            ILoggerService loggerService
+            ) : base(logFileService, loggerService)
+        {
+            // do nothing
+        }
+
+        public override void Schedule()
+        {
+            loggerService.StartMethod();
+
+            WorkManager workManager = WorkManager.GetInstance(Platform.AppContext);
+
+            PeriodicWorkRequest periodicWorkRequest = CreatePeriodicWorkRequest();
+            workManager.EnqueueUniquePeriodicWork(
+                CURRENT_WORK_NAME,
+                ExistingPeriodicWorkPolicy.Keep,
+                periodicWorkRequest
+                );
+
+            loggerService.EndMethod();
+        }
+
+        private PeriodicWorkRequest CreatePeriodicWorkRequest()
+        {
+            var workRequestBuilder = new PeriodicWorkRequest.Builder(
+                typeof(BackgroundWorker),
+                INTERVAL_IN_HOURS, TimeUnit.Hours
+                )
+                .SetConstraints(new Constraints.Builder()
+                   .SetRequiresBatteryNotLow(true)
+                   .Build())
+                .SetBackoffCriteria(BackoffPolicy.Linear, BACKOFF_DELAY_IN_MINUTES, TimeUnit.Minutes);
+            return workRequestBuilder.Build();
+        }
+    }
+
+    [Preserve]
+    public class BackgroundWorker : Worker
+    {
+        private Lazy<AbsDataMaintainanceBackgroundService> _dataMaintainanceBackgroundService
+            => new Lazy<AbsDataMaintainanceBackgroundService>(() => ContainerLocator.Current.Resolve<AbsDataMaintainanceBackgroundService>());
+        private Lazy<ILoggerService> _loggerService => new Lazy<ILoggerService>(() => ContainerLocator.Current.Resolve<ILoggerService>());
+
+        public BackgroundWorker(Context context, WorkerParameters workerParameters)
+            : base(context, workerParameters)
+        {
+            // do nothing
+        }
+
+        public override Result DoWork()
+        {
+            var dataMaintainanceBackgroundService = _dataMaintainanceBackgroundService.Value;
+            var loggerService = _loggerService.Value;
+
+            loggerService.StartMethod();
+
+            try
+            {
+                dataMaintainanceBackgroundService.ExecuteAsync().GetAwaiter().GetResult();
+                return Result.InvokeSuccess();
+            }
+            catch (Exception exception)
+            {
+                loggerService.Exception("Exception", exception);
+                return Result.InvokeFailure();
+            }
+            finally
+            {
+                loggerService.EndMethod();
+            }
+        }
+
+        public override void OnStopped()
+        {
+            base.OnStopped();
+
+            _loggerService.Value.Warning("OnStopped");
+        }
+
+    }
+}

Covid19Radar/Covid19Radar.Android/Services/EventLogSubmissionBackgroundService.cs

@@ -17,7 +17,7 @@ namespace Covid19Radar.Droid.Services
 {
     public class EventLogSubmissionBackgroundService : AbsEventLogSubmissionBackgroundService
     {
-        private const string CURRENT_WORK_NAME = "eventlog_submission_worker_20220112";
+        private const string CURRENT_WORK_NAME = "eventlog_submission_worker_20220628";
 
         private const long INTERVAL_IN_HOURS = 24;
         private const long BACKOFF_DELAY_IN_MINUTES = 60;
@@ -40,14 +40,14 @@ public override void Schedule()
             PeriodicWorkRequest periodicWorkRequest = CreatePeriodicWorkRequest();
             workManager.EnqueueUniquePeriodicWork(
                 CURRENT_WORK_NAME,
-                ExistingPeriodicWorkPolicy.Replace,
+                ExistingPeriodicWorkPolicy.Keep,
                 periodicWorkRequest
                 );
 
             _loggerService.EndMethod();
         }
 
-        private static PeriodicWorkRequest CreatePeriodicWorkRequest()
+        private PeriodicWorkRequest CreatePeriodicWorkRequest()
         {
             var workRequestBuilder = new PeriodicWorkRequest.Builder(
                 typeof(BackgroundWorker),

Covid19Radar/Covid19Radar.Android/Services/Migration/MigrationProcessService.cs

@@ -69,14 +69,20 @@ public override Result DoWork()
     public class MigrationProccessService : IMigrationProcessService
     {
         private readonly AbsExposureDetectionBackgroundService _exposureDetectionBackgroundService;
+        private readonly AbsDataMaintainanceBackgroundService _dataMaintainanceBackgroundService;
+        private readonly AbsEventLogSubmissionBackgroundService _eventLogSubmissionBackgroundService;
         private readonly ILoggerService _loggerService;
 
         public MigrationProccessService(
             AbsExposureDetectionBackgroundService exposureDetectionBackgroundService,
+            AbsDataMaintainanceBackgroundService dataMaintainanceBackgroundService,
+            AbsEventLogSubmissionBackgroundService eventLogSubmissionBackgroundService,
             ILoggerService loggerService
             )
         {
             _exposureDetectionBackgroundService = exposureDetectionBackgroundService;
+            _dataMaintainanceBackgroundService = dataMaintainanceBackgroundService;
+            _eventLogSubmissionBackgroundService = eventLogSubmissionBackgroundService;
             _loggerService = loggerService;
         }
 
@@ -86,6 +92,8 @@ public async Task SetupAsync()
 
             await new WorkManagerMigrator(
                 _exposureDetectionBackgroundService,
+                _dataMaintainanceBackgroundService,
+                _eventLogSubmissionBackgroundService,
                 _loggerService
                 ).ExecuteAsync();
 

Covid19Radar/Covid19Radar.Android/Services/Migration/WorkManagerMigrator.cs

@@ -19,14 +19,21 @@ internal class WorkManagerMigrator
         };
 
         private readonly AbsExposureDetectionBackgroundService _exposureDetectionBackgroundService;
+        private readonly AbsDataMaintainanceBackgroundService _dataMaintainanceBackgroundService;
+        private readonly AbsEventLogSubmissionBackgroundService _eventLogSubmissionBackgroundService;
+
         private readonly ILoggerService _loggerService;
 
         public WorkManagerMigrator(
             AbsExposureDetectionBackgroundService exposureDetectionBackgroundService,
+            AbsDataMaintainanceBackgroundService dataMaintainanceBackgroundService,
+            AbsEventLogSubmissionBackgroundService eventLogSubmissionBackgroundService,
             ILoggerService loggerService
             )
         {
             _exposureDetectionBackgroundService = exposureDetectionBackgroundService;
+            _dataMaintainanceBackgroundService = dataMaintainanceBackgroundService;
+            _eventLogSubmissionBackgroundService = eventLogSubmissionBackgroundService;
             _loggerService = loggerService;
         }
 
@@ -38,6 +45,8 @@ internal Task ExecuteAsync()
             CancelOldWorks(workManager, OldWorkNames, _loggerService);
 
             _exposureDetectionBackgroundService.Schedule();
+            _dataMaintainanceBackgroundService.Schedule();
+            _eventLogSubmissionBackgroundService.Schedule();
 
             _loggerService.EndMethod();
 

Covid19Radar/Covid19Radar.iOS/AppDelegate.cs

@@ -53,6 +53,9 @@ private Lazy<ILoggerService> _loggerService
         private Lazy<IExposureConfigurationRepository> _exposureConfigurationRepository
             = new Lazy<IExposureConfigurationRepository>(() => ContainerLocator.Current.Resolve<IExposureConfigurationRepository>());
 
+        private Lazy<AbsDataMaintainanceBackgroundService> _dataMaintainanceBackgroundService
+            = new Lazy<AbsDataMaintainanceBackgroundService>(() => ContainerLocator.Current.Resolve<AbsDataMaintainanceBackgroundService>());
+
         private App? AppInstance
         {
             get
@@ -112,12 +115,12 @@ public override bool FinishedLaunching(UIApplication app, NSDictionary launchOpt
 
             UIApplication.SharedApplication.SetMinimumBackgroundFetchInterval(UIApplication.BackgroundFetchIntervalMinimum);
 
-            ScheduleBackgroundTask();
+            ScheduleBackgroundTasks();
 
             return base.FinishedLaunching(app, launchOptions);
         }
 
-        private void ScheduleBackgroundTask()
+        private void ScheduleBackgroundTasks()
         {
             try
             {
@@ -127,6 +130,7 @@ private void ScheduleBackgroundTask()
             {
                 _loggerService.Value.Exception("Failed to schedule ExposureDetectionBackgroundService", exception);
             }
+
             try
             {
                 _eventLogSubmissionBackgroundService.Value.Schedule();
@@ -135,6 +139,15 @@ private void ScheduleBackgroundTask()
             {
                 _loggerService.Value.Exception("Failed to schedule EventLogSubmissionBackgroundService", exception);
             }
+
+            try
+            {
+                _dataMaintainanceBackgroundService.Value.Schedule();
+            }
+            catch (Exception exception)
+            {
+                _loggerService.Value.Exception("Failed to schedule DataMaintainanceBackgroundService", exception);
+            }
         }
 
         private bool IsUniversalLinks(NSDictionary launchOptions)
@@ -256,7 +269,7 @@ private void RegisterPlatformTypes(IContainer container)
             // Services
             container.Register<IBackupAttributeService, BackupAttributeService>(Reuse.Singleton);
             container.Register<ILocalPathService, LocalPathService>(Reuse.Singleton);
-            container.Register<ILogPeriodicDeleteService, LogPeriodicDeleteService>(Reuse.Singleton);
+            container.Register<AbsDataMaintainanceBackgroundService, DataMaintainanceBackgroundService>(Reuse.Singleton);
             container.Register<ISecureStorageDependencyService, Services.SecureStorageService>(Reuse.Singleton);
             container.Register<IPreferencesService, PreferencesService>(Reuse.Singleton);
             container.Register<IApplicationPropertyService, ApplicationPropertyService>(Reuse.Singleton);