better handling of exceptions, again

YunoHost · Sep 14, 2024 · e08d9dd · e08d9dd
1 parent b2bb20a
commit e08d9dd
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/autoupdate_app_sources/rest_api.py b/autoupdate_app_sources/rest_api.py
@@ -66,6 +66,7 @@ def changelog_for_ref(self, new_ref: str, old_ref: str, ref_type: RefType) -> st
 class GitlabAPI:
     def __init__(self, upstream: str):
         # Find gitlab api root...
+        upstream = upstream.rstrip("/")
         self.forge_root = self.get_forge_root(upstream).rstrip("/")
         self.project_path = upstream.replace(self.forge_root, "").lstrip("/")
         self.project_id = self.find_project_id(self.project_path)

diff --git a/find_deprecated.py b/find_deprecated.py
@@ -60,12 +60,16 @@ def upstream_last_update_ago(app: str) -> tuple[str, int | None]:
     api = None
     try:
         if upstream.startswith("https://github.com/"):
-            api = GithubAPI(upstream, auth=get_github()[0])
+            try:
+                api = GithubAPI(upstream, auth=get_github()[0])
+            except AssertionError as e:
+                logging.error(f"Exception while handling {app}: {e}")
+                return app, None
 
-        if upstream.startswith("https://gitlab."):
+        if upstream.startswith("https://gitlab.") or upstream.startswith("https://framagit.org"):
             api = GitlabAPI(upstream)
 
-        if upstream.startswith("https://codeberg.org") or upstream.startswith("https://framagit.org"):
+        if upstream.startswith("https://codeberg.org"):
             api = GiteaForgejoAPI(upstream)
 
         if not api: