Merge branch 'main' into attributes-backfill

Author: jvyden

…s/Migrations/BackfillRevisionMigrator.cs …/Migrations/BackfillRevisionMigration.csRefresh.Interfaces.Workers/Migrations/BackfillRevisionMigrator.cs renamed to Refresh.Interfaces.Workers/Migrations/BackfillRevisionMigration.cs Refresh.Interfaces.Workers/Migrations/BackfillRevisionMigrator.cs renamed to Refresh.Interfaces.Workers/Migrations/BackfillRevisionMigration.cs

@@ -0,0 +1,26 @@
+using Refresh.Core;
+using Refresh.Database.Models.Users;
+using Refresh.Workers;
+
+namespace Refresh.Interfaces.Workers.Migrations;
+
+public class EnsureDeletedUsersDeletedMigration : MigrationJob<GameUser>
+{
+    protected override int BatchCount => 5; // lower batch size as deletion takes a while
+
+    protected override IQueryable<GameUser> SortAndFilter(IQueryable<GameUser> query)
+    {
+        return query
+            .Where(u => u.BanReason == "This user's account has been deleted." || u.PasswordBcrypt == "deleted")
+            .OrderBy(u => u.UserId); // can't use join date here as we're changing the join date when we delete data
+    }
+
+    protected override void Migrate(WorkContext context, GameUser[] batch)
+    {
+        foreach (GameUser user in batch)
+        {
+            context.Logger.LogWarning(RefreshContext.Worker, $"Deleting {user.Username}'s account again to ensure data has been wiped...");
+            context.Database.DeleteUser(user);
+        }
+    }
+}

Refresh.Interfaces.Workers/Migrations/EnsureDeletedUsersDeletedMigration.cs

@@ -20,6 +20,7 @@ public static WorkerManager Create(Logger logger, IDataStore dataStore, GameData
         manager.AddJob<ObjectStatisticsJob>();
 
         manager.AddJob<BackfillRevisionMigration>();
+        manager.AddJob<EnsureDeletedUsersDeletedMigration>();
 
         return manager;
     }