Remove 504 HTTP status code retry

reportportal · Sep 17, 2024 · da4f7b8 · da4f7b8
1 parent 389e569
commit da4f7b8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,8 @@
 # Changelog
 
 ## [Unreleased]
+### Removed
+- Retries of requests ended with `504` HTTP status code, since it's not clear if the request was delivered or not, by @HardNorth
 
 ## [5.5.7]
 ### Added

diff --git a/reportportal_client/_internal/aio/http.py b/reportportal_client/_internal/aio/http.py
@@ -33,7 +33,7 @@
 DEFAULT_RETRY_NUMBER: int = 5
 DEFAULT_RETRY_DELAY: float = 0.005
 THROTTLING_STATUSES: set = {425, 429}
-RETRY_STATUSES: set = {408, 500, 502, 503, 504, 507}.union(THROTTLING_STATUSES)
+RETRY_STATUSES: set = {408, 500, 502, 503, 507}.union(THROTTLING_STATUSES)
 
 
 class RetryClass(int, Enum):