Don't spam logs when checking for world changes

Author: VolcanicArts

VRCOSC.Game/Util/CurrentWorldExtractor.cs

@@ -24,6 +24,7 @@ public static class CurrentWorldExtractor
     private static IEnumerable<string>? ids;
 
     public static bool IsCurrentWorldBlacklisted { get; private set; }
+    public static string? CurrentWorldId { get; private set; }
 
     public static async void UpdateCurrentWorld()
     {
@@ -35,15 +36,19 @@ public static async void UpdateCurrentWorld()
             return;
         }
 
-        var currentWorldId = await getCurrentWorldId();
+        var newCurrentWorldId = await getCurrentWorldId();
+        if (newCurrentWorldId == CurrentWorldId) return;
 
-        if (currentWorldId is null)
+        CurrentWorldId = newCurrentWorldId;
+        Logger.Log($"World change detected: {CurrentWorldId}");
+
+        if (CurrentWorldId is null)
         {
             IsCurrentWorldBlacklisted = false;
             return;
         }
 
-        IsCurrentWorldBlacklisted = ids.Contains(currentWorldId);
+        IsCurrentWorldBlacklisted = ids.Contains(CurrentWorldId);
     }
 
     private static async Task requestBlacklist()
@@ -52,21 +57,15 @@ private static async Task requestBlacklist()
         {
             if (blacklist is not null) return;
 
-            Logger.Log("Updating world blacklist");
-
             using var client = new HttpClient();
 
             var response = await client.GetAsync(blacklist_url);
             response.EnsureSuccessStatusCode();
 
-            Logger.Log("Found blacklist file");
-
             var content = await response.Content.ReadAsStringAsync();
             blacklist = JsonConvert.DeserializeObject<Blacklist>(content);
             if (blacklist is null) return;
 
-            Logger.Log("Successfully extracted blacklist data");
-
             ids = blacklist.Worlds.Select(world => world.ID);
         }
         catch (Exception)
@@ -80,26 +79,19 @@ private static async Task requestBlacklist()
     {
         try
         {
-            Logger.Log($"Attempting to find current world using log pattern {logfile_location}\\{logfile_pattern}");
-
             var logFile = Directory.GetFiles(logfile_location, logfile_pattern).MaxBy(d => new FileInfo(d).CreationTime);
             if (logFile is null) return null;
 
-            Logger.Log($"Found latest log file at {logFile}");
-
             using var fileStream = new FileStream(logFile, FileMode.Open, FileAccess.Read, FileShare.ReadWrite);
             using var textReader = new StreamReader(fileStream);
             var content = await textReader.ReadToEndAsync();
 
-            Logger.Log("Successfully read latest log file");
-
             var latestWorld = world_regex.Matches(content).LastOrDefault()?.Groups.Values.LastOrDefault();
             if (latestWorld is null) return null;
 
             var latestWorldId = latestWorld.Captures.FirstOrDefault();
             if (latestWorldId is null) return null;
 
-            Logger.Log($"Found current world: {latestWorldId.Value}");
             return latestWorldId.Value;
         }
         catch (Exception)