From da4f7b83aa6424eb709763264fe51c9bd20f90a5 Mon Sep 17 00:00:00 2001
From: Vadzim Hushchanskou <vadzim_hushchanskou@epam.com>
Date: Tue, 17 Sep 2024 14:16:13 +0300
Subject: [PATCH] Remove 504 HTTP status code retry

---
 CHANGELOG.md                              | 2 ++
 reportportal_client/_internal/aio/http.py | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 3012ac1a..8f433532 100644
--- 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
index c9c6d344..d0fd70d1 100644
--- 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):