Optimize retries strategy

huangwb8 · Jun 6, 2023 · 3ac72a1 · 3ac72a1
1 parent 6acfb0b
commit 3ac72a1
Showing 1 changed file with 19 additions and 22 deletions.
diff --git a/myblog.py b/myblog.py
@@ -112,17 +112,15 @@ async def main():
                         break
                     except Exception as e:
                         print("OOPS, the REST API mode failed!")
-                        try:
-                            os.remove(path_legacy_json)
-                            os.rename(
-                                path_legacy_json + "_temporary-copy", path_legacy_json
-                            )
-                            if retry < max_retries - 1:
-                                print("Retrying...")
-                                continue
-                            else:
-                                print("Maximum retries exceeded. Exiting.")
-                        finally:
+                        os.remove(path_legacy_json)
+                        os.rename(
+                            path_legacy_json + "_temporary-copy", path_legacy_json
+                        )
+                        if retry < max_retries - 1:
+                            print("Retrying...")
+                            continue
+                        else:
+                            print("Maximum retries exceeded. Exiting.")
                             sys.exit(0)
             else:
                 if verbose:
@@ -159,18 +157,17 @@ async def main():
                         break
                     except Exception as e:
                         print("OOPS, the Password mode failed!")
-                        try:
-                            os.remove(path_legacy_json)
-                            os.rename(
-                                path_legacy_json + "_temporary-copy", path_legacy_json
-                            )
-                            if retry < max_retries - 1:
-                                print("Retrying...")
-                                continue
-                            else:
-                                print("Maximum retries exceeded. Exiting.")
-                        finally:
+                        os.remove(path_legacy_json)
+                        os.rename(
+                            path_legacy_json + "_temporary-copy", path_legacy_json
+                        )
+                        if retry < max_retries - 1:
+                            print("Retrying...")
+                            continue
+                        else:
+                            print("Maximum retries exceeded. Exiting.")
                             sys.exit(0)
+
             else:
                 if verbose:
                     print("Without any new or changed legacy markdown files. Ignored.")