use debug for printing errors instead of display (#98)

scrayosnet · Apr 27, 2024 · 5fb2e32 · 5fb2e32
1 parent dd8ac6c
commit 5fb2e32
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 8 deletions.
diff --git a/src/cache/level/redis.rs b/src/cache/level/redis.rs
@@ -63,7 +63,7 @@ impl RedisCache {
             .get(key)
             .await
             .unwrap_or_else(|err| {
-                error!("Failed to get value from redis: {}", err);
+                error!("Failed to get value from redis: {:?}", err);
                 None
             })
     }
@@ -84,7 +84,7 @@ impl RedisCache {
             )
             .await
             .unwrap_or_else(|err| {
-                error!("Failed to set value to redis: {}", err);
+                error!("Failed to set value to redis: {:?}", err);
             });
     }
 }

diff --git a/src/mojang/api.rs b/src/mojang/api.rs
@@ -77,18 +77,22 @@ impl MojangApi {
         })
         .await
         .map_err(|err| {
-            warn!("failed to fetch uuids: {}", err);
+            warn!("failed to fetch uuids: {:?}", err);
             Unavailable
         })?;
 
         match response.status() {
             StatusCode::NOT_FOUND | StatusCode::NO_CONTENT => Ok(vec![]),
             StatusCode::OK => response.json().await.map_err(|err| {
-                error!("failed to read body (username_resolved): {}", err);
+                error!("failed to read body (username_resolved): {:?}", err);
                 Unavailable
             }),
             code => {
-                warn!(status = code.as_str(), "{:?}", response.text().await.ok());
+                warn!(
+                    status = code.as_str(),
+                    "{:?}",
+                    response.text().await.unwrap_or(String::new())
+                );
                 Err(Unavailable)
             }
         }
@@ -121,14 +125,14 @@ impl Mojang for MojangApi {
         })
         .await
         .map_err(|err| {
-            warn!("failed to fetch profile: {}", err);
+            warn!("failed to fetch profile: {:?}", err);
             Unavailable
         })?;
 
         match response.status() {
             StatusCode::NOT_FOUND | StatusCode::NO_CONTENT => Err(NotFound),
             StatusCode::OK => response.json().await.map_err(|err| {
-                error!("failed to read body (profile): {}", err);
+                error!("failed to read body (profile): {:?}", err);
                 Unavailable
             }),
             code => {
@@ -150,7 +154,7 @@ impl Mojang for MojangApi {
         match response.status() {
             StatusCode::NOT_FOUND | StatusCode::NO_CONTENT => Err(NotFound),
             StatusCode::OK => response.bytes().await.map_err(|err| {
-                error!("failed to read body (bytes): {}", err);
+                error!("failed to read body (bytes): {:?}", err);
                 Unavailable
             }),
             code => {