[fix] Fixed several minor issues in retryable requests

- we need to ensure a recent version of requests is installed
- docs had several mistakes
- some code was repeated twice in the function definition

README.rst

@@ -1348,8 +1348,8 @@ Usage:
 but not for complex background tasks which can take a long time to execute
 (eg: firmware upgrades, network operations with retry mechanisms).
 
-``openwisp_utils.tasks.retryable_requests``
-^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+``openwisp_utils.utils.retryable_request``
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
 A utility function for making HTTP requests with built-in retry logic.
 This function is useful for handling transient errors encountered during HTTP
@@ -1361,7 +1361,7 @@ Usage:
 
 .. code-block:: python
 
-    from your_module import retryable_request
+    from openwisp_utils.utils import retryable_request
 
     response = retryable_request(
         method='GET',

openwisp_utils/utils.py

@@ -88,16 +88,6 @@ def retryable_request(
             backoff_jitter=backoff_jitter,
         )
     )
-    retry_kwargs = retry_kwargs or {}
-    retry_kwargs.update(
-        dict(
-            total=max_retries,
-            backoff_factor=backoff_factor,
-            backoff_jitter=backoff_jitter,
-            status_forcelist=status_forcelist,
-            allowed_methods=allowed_methods,
-        )
-    )
     request_session = requests.Session()
     retries = Retry(**retry_kwargs)
     request_session.mount('https://', HTTPAdapter(max_retries=retries))

setup.py

@@ -46,6 +46,8 @@
         'django-compress-staticfiles~=1.0.1b',
         'django-admin-autocomplete-filter~=0.7.1',
         'swapper~=1.3.0',
+        # allow wider range here to avoid interfering with other modules
+        'requests>=2.31.0,<3.0.0',
     ],
     extras_require={
         'qa': [