copyparty: never send the password to foreign hosts

1 files changed, 13 insertions, 4 deletions

diff --git a/src/voice/mod.rs b/src/voice/mod.rs
index fcb0b07..5a89f93 100644
--- a/src/voice/mod.rs
+++ b/src/voice/mod.rs
@@ -16,6 +16,7 @@ use symphonia::core::meta::MetadataOptions;
 use tokio::sync::Mutex;
 use tracing::{info, warn};
 
+use crate::copyparty::redact_url;
 use crate::error::{Error, Result};
 use crate::lang;
 use crate::recording::{Recorder, RecorderEventHandler};
@@ -241,7 +242,11 @@ pub async fn ytdlp_download(url: &str) -> Result<Vec<u8>> {
         let stderr = String::from_utf8_lossy(&output.stderr);
         // Keep the full yt-dlp trace in the logs for debugging — the user
         // only ever sees the short, translated `fmt_ytdlp_stderr` message.
-        warn!(url = %url, "yt-dlp failed:\n{}", stderr.trim_end());
+        warn!(
+            url = %redact_url(url),
+            "yt-dlp failed:\n{}",
+            stderr.trim_end()
+        );
         return Err(Error::Custom(lang::fmt_ytdlp_stderr(&stderr)));
     }
 
@@ -256,11 +261,11 @@ pub async fn download_url(client: &reqwest::Client, url: &str) -> Result<Vec<u8>
         .get(url)
         .send()
         .await
-        .map_err(|e| Error::Custom(format!("HTTP GET failed for {url}: {e}")))?;
+        .map_err(|e| Error::Custom(format!("HTTP GET failed for {}: {e}", redact_url(url))))?;
     let bytes = resp
         .bytes()
         .await
-        .map_err(|e| Error::Custom(format!("Failed to read body for {url}: {e}")))?;
+        .map_err(|e| Error::Custom(format!("Failed to read body for {}: {e}", redact_url(url))))?;
     Ok(bytes.to_vec())
 }
 
@@ -398,7 +403,11 @@ pub async fn ffmpeg_stream_input(url: &str) -> Result<songbird::input::Input> {
             let _ = reader.read_to_end(&mut buf).await;
             let text = String::from_utf8_lossy(&buf);
             if !text.is_empty() {
-                warn!(url = %url_owned, "ffmpeg stderr: {}", text.trim_end());
+                warn!(
+                    url = %redact_url(&url_owned),
+                    "ffmpeg stderr: {}",
+                    text.trim_end()
+                );
             }
         });
     }