Add more logging to repo add

DEVELOPEST · Mar 3, 2021 · 42e0608 · 42e0608
1 parent 4a71048
commit 42e0608
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/src/main.rs b/src/main.rs
@@ -1,7 +1,7 @@
 #![feature(proc_macro_hygiene, decl_macro)]
 
 #[macro_use] extern crate rocket;
-#[macro_use(info, warn)] extern crate log;
+#[macro_use(info, warn, error)] extern crate log;
 
 mod server;
 mod gtm;

diff --git a/src/repo/repo_manager.rs b/src/repo/repo_manager.rs
@@ -104,11 +104,13 @@ pub async fn add_repo(repo_dto: AddRepositoryDto) -> AddRepoDto {
             message: None,
         };
     }
+    let error_msg = cloned_repo.err().map(|e| e.to_string()).unwrap_or_default();
+    error!("Error adding repo: {}", &error_msg);
     return AddRepoDto {
         success: false,
         provider: None,
         user: None,
         repo: None,
-        message: cloned_repo.err().map(|e| e.to_string()),
+        message: Option::from(error_msg),
     };
 }
diff --git a/src/sync/sync.rs b/src/sync/sync.rs
@@ -63,6 +63,7 @@ pub async fn sync_repo(provider: &String, user: &String, repo: &String) -> SyncS
     let res = sync_single(&repo_to_sync, &cfg, &client).await;
     // TODO: Check for error in json
     if res.is_err() || !res.unwrap().status().is_success() {
+        error!("Error syncing repo!");
         return SyncSingleResult { error: Option::from("Error syncing repo!".to_string()), ok: false }
     }
     return SyncSingleResult { error: None, ok: true }