Merge pull request #121 from nullinside-development-group/feat/skip

fix: crash when skipping messages in deployed version

Author: ProgrammingByPermutation

src/TwitchStreamingTools/Services/TwitchTtsService.cs

@@ -93,8 +93,14 @@ private void OnKeystroke(Keybind keybind) {
         keybind.IsAlt == skip.IsAlt &&
         keybind.IsCtrl == skip.IsCtrl &&
         keybind.IsShift == skip.IsShift) {
-      foreach (TwitchChatTts chat in _chats) {
-        chat.SkipCurrentTts();
+      var chats = _chats.ToList();
+      foreach (TwitchChatTts chat in chats) {
+        try {
+          chat.SkipCurrentTts();
+        }
+        catch {
+          // Do nothing, just try to skip the best we can.
+        }
       }
     }
   }

src/TwitchStreamingTools/ViewModels/Pages/SettingsView/TtsSkipUsernamesViewModel.cs

@@ -182,9 +182,8 @@ private async void UserListRefreshTimer_OnElapsed(object sender, ElapsedEventArg
       _userListRefreshTimer.Interval = 5000;
       _userListRefreshTimer.Start();
     }
-    catch (Exception ex) {
+    catch {
       // do nothing don't crash
-      LOG.Error("Failed to refresh user list", ex);
     }
   }
 }