fix: various improvements to api service

api/src/api/package.rs

@@ -943,10 +943,11 @@ pub async fn version_provenance_statements_handler(
     .await?;
 
   if let Some(orama_client) = orama_client {
-    let (package, _, meta) = db
-      .get_package(&scope, &package)
-      .await?
-      .ok_or(ApiError::InternalServerError)?;
+    let (package, _, meta) =
+      db.get_package(&scope, &package).await?.ok_or_else(|| {
+        error!("package not found after inserting provenance statement");
+        ApiError::InternalServerError
+      })?;
     orama_client.upsert_package(&package, &meta);
   }
 

api/src/api/scope.rs

@@ -14,6 +14,7 @@ use hyper::StatusCode;
 use routerify::Router;
 use routerify::ext::RequestExt;
 use tracing::Span;
+use tracing::error;
 use tracing::field;
 use tracing::instrument;
 
@@ -385,10 +386,13 @@ async fn update_member_handler(
     }
   };
 
-  let user = db
-    .get_user_public(scope_member.user_id)
-    .await?
-    .ok_or(ApiError::InternalServerError)?;
+  let user =
+    db.get_user_public(scope_member.user_id)
+      .await?
+      .ok_or_else(|| {
+        error!("user not found for scope member: {}", scope_member.user_id);
+        ApiError::InternalServerError
+      })?;
 
   Ok((scope_member, user).into())
 }

api/src/main.rs

@@ -146,7 +146,7 @@ async fn main() {
   let database = Database::connect(
     &config.database_url,
     config.database_pool_size,
-    Duration::from_secs(5),
+    Duration::from_secs(15),
   )
   .await
   .unwrap();

api/src/publish.rs

@@ -144,7 +144,13 @@ pub async fn publish_task(
               &publishing_task.package_name,
             )
             .await?
-            .ok_or(ApiError::InternalServerError)?;
+            .ok_or_else(|| {
+              error!(
+                "package not found after successful publishing: {}/{}",
+                &publishing_task.package_scope, &publishing_task.package_name
+              );
+              ApiError::InternalServerError
+            })?;
           orama_client.upsert_package(&package, &meta);
         }
         return Ok(());