fixes empty api search results (#115)

src/auto_archiver/core/media.py

@@ -65,7 +65,9 @@ class Media:
 
     @property  # getter .mimetype
     def mimetype(self) -> str:
-        assert self.filename is not None and len(self.filename) > 0, "cannot get mimetype from media without filename"
+        if not self.filename or len(self.filename) == 0:
+            logger.warning(f"cannot get mimetype from media without filename: {self}")
+            return ""
         if not self._mimetype:
             self._mimetype = mimetypes.guess_type(self.filename)[0]
         return self._mimetype or ""

src/auto_archiver/databases/api_db.py

@@ -40,11 +40,12 @@ class AAApiDb(Database):
         response = requests.get(os.path.join(self.api_endpoint, "tasks/search-url"), params=params, headers=headers)
 
         if response.status_code == 200:
-            logger.success(f"API returned a previously archived instance: {response.json()}")
-            # TODO: can we do better than just returning the first result?
-            return Metadata.from_dict(response.json()[0]["result"])
-        
-        logger.error(f"AA API FAIL ({response.status_code}): {response.json()}")
+            if len(response.json()):
+                logger.success(f"API returned a previously archived instance: {response.json()}")
+                # TODO: can we do better than just returning the most recent result?
+                return Metadata.from_dict(response.json()[0]["result"])
+        else:
+            logger.error(f"AA API FAIL ({response.status_code}): {response.json()}")
         return False
 
 

src/auto_archiver/version.py

@@ -3,7 +3,7 @@ _MAJOR = "0"
 _MINOR = "7"
 # On main and in a nightly release the patch should be one ahead of the last
 # released build.
-_PATCH = "2"
+_PATCH = "4"
 # This is mainly for nightly builds which have the suffix ".dev$DATE". See
 # https://semver.org/#is-v123-a-semantic-version for the semantics.
 _SUFFIX = ""