Merge pull request #2676 from locustio/update-geventhttpclient-max_re…

…quests-and-max_retries-behaviour Update geventhttpclient and adjust FastHttpUser max_retries / max_redirects
locustio · Apr 17, 2024 · 6ddfc61 · 6ddfc61
2 parents 680ec6b + 417df8a
commit 6ddfc61
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/locust/contrib/fasthttp.py b/locust/contrib/fasthttp.py
@@ -306,11 +306,11 @@ class FastHttpUser(User):
     connection_timeout: float = 60.0
     """Parameter passed to FastHttpSession"""
 
-    max_redirects: int = 5
-    """Parameter passed to FastHttpSession. Default 5, meaning 4 redirects."""
+    max_redirects: int = 30
+    """Parameter passed to FastHttpSession."""
 
-    max_retries: int = 1
-    """Parameter passed to FastHttpSession. Default 1, meaning zero retries."""
+    max_retries: int = 0
+    """Parameter passed to FastHttpSession."""
 
     insecure: bool = True
     """Parameter passed to FastHttpSession. Default True, meaning no SSL verification."""

diff --git a/locust/test/test_fasthttp.py b/locust/test/test_fasthttp.py
@@ -506,7 +506,7 @@ class MyUser(FastHttpUser):
 
     def test_max_redirect_setting(self):
         class MyUser(FastHttpUser):
-            max_redirects = 1  # max_redirects and max_retries are funny names, because they are actually max attempts
+            max_redirects = 0
             host = "http://127.0.0.1:%i" % self.port
 
         l = MyUser(self.environment)

diff --git a/pyproject.toml b/pyproject.toml
@@ -15,7 +15,7 @@ dependencies = [
     "requests >=2.26.0",
     "msgpack >=1.0.0",
     "pyzmq >=25.0.0",
-    "geventhttpclient >=2.0.11",
+    "geventhttpclient >=2.2.1",
     "ConfigArgParse >=1.5.5",
     "tomli >=1.1.0; python_version<'3.11'",
     "psutil >=5.9.1",