Allow packages through which cause metadata injector to raise (e.g. /browse/project/distance/0.1.3/files)

Author: pelson

simple_repository_browser/fetch_description.py

@@ -15,7 +15,9 @@
 import readme_renderer.markdown
 import readme_renderer.rst
 import readme_renderer.txt
+import simple_repository
 from simple_repository import SimpleRepository, model
+import simple_repository.errors
 
 
 @dataclasses.dataclass
@@ -153,6 +155,13 @@ def _create_files_info_mapping(
     return files_info
 
 
+class MinimalDistribution(pkginfo.Distribution):
+    def __init__(self, name: str, summary: str):
+        super().__init__()  # Get all the default None values
+        self.name = name
+        self.summary = summary
+
+
 async def _fetch_metadata_resource(
     repository: SimpleRepository,
     project_name: str,
@@ -163,21 +172,22 @@ async def _fetch_metadata_resource(
     if not file.dist_info_metadata:
         # No metadata available for this file type (e.g., .egg files)
         # Return a minimal distribution object with basic info
-        class MinimalDistribution(pkginfo.Distribution):
-            def __init__(self, name: str, filename: str):
-                super().__init__()  # Get all the default None values
-                self.name = name
-                ext = filename.split(".")[-1] if "." in filename else "unknown"
-                self.summary = f"Legacy package format ({ext}) - metadata not available"
-
-        minimal_dist = MinimalDistribution(project_name, file.filename)
+        ext = file.filename.rsplit(".", 1)[-1]
+        reason = f"Legacy package format ({ext}) - metadata not available"
+        minimal_dist = MinimalDistribution(project_name, summary=reason)
         return file, minimal_dist
 
     resource_name = file.filename + ".metadata"
 
     logging.debug(f"Downloading metadata for {file.filename} from {resource_name}")
 
-    resource = await repository.get_resource(project_name, resource_name)
+    try:
+        resource = await repository.get_resource(project_name, resource_name)
+    except simple_repository.errors.ResourceUnavailable as err:
+        reason = f"Unable to retrieve metadata for {file.filename} ({err})"
+        minimal_dist = MinimalDistribution(project_name, summary=reason)
+        logging.exception(reason)
+        return file, minimal_dist
 
     if isinstance(resource, model.TextResource):
         with open(tmp_file_path, "wb") as tmp: