Don't retry adding users if they are already in the channel. (#4906)

Netflix · Jul 2, 2024 · 1a0c241 · 1a0c241
1 parent 9d1f892
commit 1a0c241
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/dispatch/plugins/dispatch_slack/service.py b/src/dispatch/plugins/dispatch_slack/service.py
@@ -103,6 +103,9 @@ def should_retry(exception: Exception) -> bool:
     """
     match exception:
         case SlackApiError():
+            # Don't retry for re-adding users in channel.
+            if exception.response["error"] == SlackAPIErrorCode.USER_IN_CHANNEL:
+                return False
             # Retry if it's not a fatal error
             return exception.response["error"] != SlackAPIErrorCode.FATAL_ERROR
         case TimeoutError() | Timeout():