Add proper logging into s3 functions

Author: a-gonzalo

src/webserver/database/sqlpage_functions/s3.rs

@@ -27,12 +27,19 @@ pub(super) async fn upload_to_s3<'a>(
         if !full_path.starts_with(web_root) {
              anyhow::bail!("Security violation: Access denied to file outside web root");
         }
-        tokio::fs::read(&full_path).await.with_context(|| format!("Unable to read file {}", full_path.display()))?
+        tokio::fs::read(&full_path).await.map_err(|e| {
+            log::error!("Failed to read file {}: {}", full_path.display(), e);
+            e
+        }).with_context(|| format!("Unable to read file {}", full_path.display()))?
     } else {
         // Assume base64
         use base64::Engine;
         base64::engine::general_purpose::STANDARD
             .decode(data.as_bytes())
+            .map_err(|e| {
+                log::error!("Base64 decode failed: {}", e);
+                e
+            })
             .context("Invalid base64 data")?
     };