Merge pull request #1118 from cocoa-mhlw/feature/bug-ios-crash

iOSで起動時にクラッシュすることがある不具合の対応

Covid19Radar/Covid19Radar.iOS/AppDelegate.cs

@@ -121,15 +121,6 @@ public override bool FinishedLaunching(UIApplication app, NSDictionary launchOpt
  // https://developer.apple.com/documentation/backgroundtasks/bgtaskscheduler/3180427-register
  ScheduleBackgroundTasks();
 
- try
- {
- LoggingPendingTaskRequests();
- }
- catch (Exception ex)
- {
- _loggerService.Value.Exception("Failure get pending task requests", ex);
- }
-
  return base.FinishedLaunching(app, launchOptions);
  }
 
@@ -167,20 +158,6 @@ private void ScheduleBackgroundTasks()
  _loggerService.Value.EndMethod();
  }
 
- private void LoggingPendingTaskRequests()
- {
- _loggerService.Value.Info($"Get pending task requests");
- BGTaskScheduler.Shared.GetPending(pendingTaskRequests =>
- {
- _loggerService.Value.Info($"Pending task count: {pendingTaskRequests.Length}");
- foreach (var pendingTaskRequest in pendingTaskRequests)
- {
- string identifier = pendingTaskRequest.Identifier.Split(".")?.Last();
- _loggerService.Value.Info($"Identifier: {identifier}, EarliestBeginDate: {pendingTaskRequest.EarliestBeginDate}");
- }
- });
- }
-
  private bool IsUniversalLinks(NSDictionary launchOptions)
  {
  if (launchOptions == null)
@@ -293,15 +270,6 @@ public override void OnActivated(UIApplication uiApplication)
  {
  base.OnActivated(uiApplication);
  MessagingCenter.Send((object)this, AppConstants.IosOnActivatedMessage);
-
- try
- {
- LoggingPendingTaskRequests();
- }
- catch (Exception ex)
- {
- _loggerService.Value.Exception("Failure get pending task requests", ex);
- }
  }
 
  private void RegisterPlatformTypes(IContainer container)

Covid19Radar/Covid19Radar.iOS/Covid19Radar.iOS.csproj

@@ -271,6 +271,7 @@
  <Compile Include="Services\LocalPathService.cs" />
  <Compile Include="Services\EventLogSubmissionBackgroundService.cs" />
  <Compile Include="Services\DataMaintainanceBackgroundService.cs" />
+ <Compile Include="Services\Migration\BGTaskMigrator.cs" />
  </ItemGroup>
  <ItemGroup>
  <ImageAsset Include="Assets.xcassets\Contents.json">

Covid19Radar/Covid19Radar.iOS/Services/Migration/BGTaskMigrator.cs

@@ -0,0 +1,43 @@
+// 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.Threading.Tasks;
+using BackgroundTasks;
+using Covid19Radar.Services.Logs;
+using Xamarin.Essentials;
+
+namespace Covid19Radar.iOS.Services.Migration
+{
+ internal class BGTaskMigrator
+ {
+ // Array of old identifier.
+ private static readonly string[] OLD_IDENTIFIER_ARRAY = {
+ AppInfo.PackageName + ".delete-old-logs",
+ };
+
+ private readonly ILoggerService _loggerService;
+
+ public BGTaskMigrator(
+ ILoggerService loggerService
+ )
+ {
+ _loggerService = loggerService;
+ }
+
+ internal Task ExecuteAsync()
+ {
+ _loggerService.StartMethod();
+
+ foreach (var identifier in OLD_IDENTIFIER_ARRAY)
+ {
+ BGTaskScheduler.Shared.Cancel(identifier);
+ _loggerService.Info($"BGTask {identifier} has been canceled.");
+ }
+
+ _loggerService.EndMethod();
+
+ return Task.CompletedTask;
+ }
+ }
+}

Covid19Radar/Covid19Radar.iOS/Services/Migration/MigrationProcessService.cs

@@ -2,12 +2,32 @@
  * 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 Covid19Radar.Services.Logs;
+using System.Threading.Tasks;
 using Covid19Radar.Services.Migration;
 
 namespace Covid19Radar.iOS.Services.Migration
 {
  public class MigrationProcessService : IMigrationProcessService
  {
- // Currently, It is not needed that iOS platform specific migration process.
+ private readonly ILoggerService _loggerService;
+
+ public MigrationProcessService(
+ ILoggerService loggerService
+ )
+ {
+ _loggerService = loggerService;
+ }
+
+ public async Task SetupAsync()
+ {
+ _loggerService.StartMethod();
+
+ await new BGTaskMigrator(
+ _loggerService
+ ).ExecuteAsync();
+
+ _loggerService.EndMethod();
+ }
  }
 }