switch download error logging to debug

getsentry · Aug 4, 2021 · 5759eb0 · 5759eb0
1 parent bd26c77
commit 5759eb0
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/crates/symbolicator/src/services/download/gcs.rs b/crates/symbolicator/src/services/download/gcs.rs
@@ -257,15 +257,15 @@ impl GcsDownloader {
                     super::download_stream(source, stream, destination, timeout).await
                 // If it's a client error, chances are either it's a 404 or it's permission-related.
                 } else if response.status().is_client_error() {
-                    log::trace!(
+                    log::debug!(
                         "Unexpected client error status code from GCS {} (from {}): {}",
                         &key,
                         &bucket,
                         response.status()
                     );
                     Ok(DownloadStatus::NotFound)
                 } else {
-                    log::trace!(
+                    log::debug!(
                         "Unexpected status code from GCS {} (from {}): {}",
                         &key,
                         &bucket,
@@ -277,7 +277,7 @@ impl GcsDownloader {
                 }
             }
             Ok(Err(e)) => {
-                log::trace!(
+                log::debug!(
                     "Skipping response from GCS {} (from {}): {} ({:?})",
                     &key,
                     &bucket,

diff --git a/crates/symbolicator/src/services/download/http.rs b/crates/symbolicator/src/services/download/http.rs
@@ -114,14 +114,14 @@ impl HttpDownloader {
                     super::download_stream(source, stream, destination, timeout).await
                 // If it's a client error, chances are either it's a 404 or it's permission-related.
                 } else if response.status().is_client_error() {
-                    log::trace!(
+                    log::debug!(
                         "Unexpected client error status code from {}: {}",
                         download_url,
                         response.status()
                     );
                     Ok(DownloadStatus::NotFound)
                 } else {
-                    log::trace!(
+                    log::debug!(
                         "Unexpected status code from {}: {}",
                         download_url,
                         response.status()

diff --git a/crates/symbolicator/src/services/download/mod.rs b/crates/symbolicator/src/services/download/mod.rs
@@ -140,10 +140,10 @@ impl DownloadService {
             Ok(status) => {
                 match status {
                     DownloadStatus::Completed => {
-                        log::debug!("Did not fetch debug file from {:?}: {:?}", source, status);
+                        log::debug!("Fetched debug file from {:?}: {:?}", source, status);
                     }
                     DownloadStatus::NotFound => {
-                        log::debug!("Fetched debug file from {:?}: {:?}", source, status);
+                        log::debug!("Did not fetch debug file from {:?}: {:?}", source, status);
                     }
                 };
                 Ok(status)

diff --git a/crates/symbolicator/src/services/download/sentry.rs b/crates/symbolicator/src/services/download/sentry.rs
@@ -285,14 +285,14 @@ impl SentryDownloader {
                     super::download_stream(source, stream, destination, timeout).await
                 // If it's a client error, chances are either it's a 404 or it's permission-related.
                 } else if response.status().is_client_error() {
-                    log::trace!(
+                    log::debug!(
                         "Unexpected client error status code from {}: {}",
                         download_url,
                         response.status()
                     );
                     Ok(DownloadStatus::NotFound)
                 } else {
-                    log::trace!(
+                    log::debug!(
                         "Unexpected status code from {}: {}",
                         download_url,
                         response.status()
@@ -303,7 +303,7 @@ impl SentryDownloader {
                 }
             }
             Ok(Err(e)) => {
-                log::trace!("Skipping response from {}: {}", download_url, e);
+                log::debug!("Skipping response from {}: {}", download_url, e);
                 Err(DownloadError::Reqwest(e)) // must be wrong type
             }
             // Timed out