chore: reduce unnecessary GitHub pull backtrace log

Signed-off-by: Wei Zhang <[email protected]>
TabbyML · Dec 11, 2024 · 7d21927 · 7d21927
1 parent b3a2b42
commit 7d21927
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/ee/tabby-webserver/src/service/background_job/third_party_integration/error.rs b/ee/tabby-webserver/src/service/background_job/third_party_integration/error.rs
@@ -6,6 +6,11 @@ pub fn octocrab_error_message(err: octocrab::Error) -> String {
             format!("GitHub error: {} {}", source.status_code, source.message)
         }
 
+        // no need to print the Json error backtrace
+        octocrab::Error::Json { source, .. } => {
+            format!("Json error: {}", source)
+        }
+
         // the other errors have impl Display or Debug
         _ => err.to_string(),
     }

diff --git a/ee/tabby-webserver/src/service/background_job/third_party_integration/pulls.rs b/ee/tabby-webserver/src/service/background_job/third_party_integration/pulls.rs
@@ -58,7 +58,7 @@ pub async fn list_github_pulls(
                             profile.email
                         }
                         Err(e) => {
-                            debug!("Failed to fetch user profile for {}: {}", author, e);
+                            debug!("Failed to fetch user profile for {}: {}", author, octocrab_error_message(e));
                             None
                         }
                     }