remove redundant retry logic

requests.session already has built-in retry mechanism

wikiteam3/dumpgenerator/api.py

@@ -102,20 +102,13 @@ def mwGetAPIAndIndex(url="", session=None):
     return api, index
 
 
-def checkRetryAPI(api=None, retries=5, apiclient=False, session=None):
+def checkRetryAPI(api=None, apiclient=False, session=None):
     """Call checkAPI and mwclient if necessary"""
-    retry = 0
-    retrydelay = 20
     check = None
-    while retry < retries:
-        try:
-            check = checkAPI(api, session=session)
-            break
-        except requests.exceptions.ConnectionError as e:
-            print("Connection error: %s" % (str(e)))
-            retry += 1
-            print("Start retry attempt %d in %d seconds." % (retry + 1, retrydelay))
-            time.sleep(retrydelay)
+    try:
+        check = checkAPI(api, session=session)
+    except requests.exceptions.ConnectionError as e:
+        print("Connection error: %s" % (str(e)))
 
     if check and apiclient:
         apiurl = urlparse(api)

wikiteam3/dumpgenerator/cli.py

@@ -187,7 +187,6 @@ def getParameters(params=[]):
     if api:
         check, checkedapi = checkRetryAPI(
             api=api,
-            retries=int(args.retries),
             apiclient=args.xmlrevisions,
             session=session,
         )