From a9bfee44610e859c8caaaa82d4ea3dfb8fbd10b9 Mon Sep 17 00:00:00 2001
From: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Date: Fri, 31 Jul 2026 05:05:05 +0000
Subject: [PATCH] fix(rust): reuse reqwest URL parser

Co-Authored-By: Ishaan Jaffer <155045088+ishaan-berri@users.noreply.github.com>
---
 litellm-rust/Cargo.lock                       |  1 -
 litellm-rust/Cargo.toml                       |  1 -
 litellm-rust/crates/core/Cargo.toml           |  1 -
 .../crates/core/src/logging/redaction.rs      | 29 +++----------------
 4 files changed, 4 insertions(+), 28 deletions(-)

diff --git a/litellm-rust/Cargo.lock b/litellm-rust/Cargo.lock
index ed5bc49b532..4f4aedb7624 100644
--- a/litellm-rust/Cargo.lock
+++ b/litellm-rust/Cargo.lock
@@ -1263,7 +1263,6 @@ dependencies = [
  "sha2 0.10.9",
  "thiserror 2.0.19",
  "tokio",
- "url",
 ]
 
 [[package]]
diff --git a/litellm-rust/Cargo.toml b/litellm-rust/Cargo.toml
index 9b4c93bb9ef..845464b4d60 100644
--- a/litellm-rust/Cargo.toml
+++ b/litellm-rust/Cargo.toml
@@ -31,4 +31,3 @@ futures-util = { version = "0.3", default-features = false, features = ["sink",
 base64 = "0.22"
 bytes = "1"
 colored_json = "5.0"
-url = "2.5"
diff --git a/litellm-rust/crates/core/Cargo.toml b/litellm-rust/crates/core/Cargo.toml
index 51700143509..e63dc9de43f 100644
--- a/litellm-rust/crates/core/Cargo.toml
+++ b/litellm-rust/crates/core/Cargo.toml
@@ -10,7 +10,6 @@ rand.workspace = true
 reqwest.workspace = true
 bytes.workspace = true
 futures-util.workspace = true
-url.workspace = true
 serde.workspace = true
 serde_json.workspace = true
 colored_json.workspace = true
diff --git a/litellm-rust/crates/core/src/logging/redaction.rs b/litellm-rust/crates/core/src/logging/redaction.rs
index 8019bb6f56c..4445df257d0 100644
--- a/litellm-rust/crates/core/src/logging/redaction.rs
+++ b/litellm-rust/crates/core/src/logging/redaction.rs
@@ -1,5 +1,6 @@
 use std::collections::BTreeMap;
 
+use reqwest::Url;
 use serde_json::{Map, Value};
 
 pub const PROVIDER_DEBUG_BODY_MAX_BYTES: usize = 64 * 1024;
@@ -44,7 +45,7 @@ pub fn redact_headers(headers: &[(String, String)]) -> BTreeMap<String, String>
 }
 
 pub fn redact_url(url: &str) -> String {
-    let Ok(mut parsed) = url::Url::parse(url) else {
+    let Ok(mut parsed) = Url::parse(url) else {
         return url.to_string();
     };
     if !parsed.username().is_empty() {
@@ -76,7 +77,7 @@ fn redact_value(value: Value) -> Value {
         Value::Object(map) => Value::Object(
             map.into_iter()
                 .map(|(key, value)| {
-                    if is_secret_key(&key) {
+                    if is_credential_name(&key) {
                         (key, Value::String("[REDACTED]".to_string()))
                     } else {
                         (key, redact_value(value))
@@ -89,10 +90,6 @@ fn redact_value(value: Value) -> Value {
     }
 }
 
-fn is_secret_key(key: &str) -> bool {
-    is_credential_name(key)
-}
-
 fn is_credential_name(name: &str) -> bool {
     let normalized = name
         .chars()
@@ -101,25 +98,7 @@ fn is_credential_name(name: &str) -> bool {
         .collect::<String>();
     matches!(
         normalized.as_str(),
-        "authorization"
-            | "proxyauthorization"
-            | "xapikey"
-            | "apikey"
-            | "xamzsecuritytoken"
-            | "cookie"
-            | "setcookie"
-            | "xamzsignature"
-            | "xamzcredential"
-            | "key"
-            | "accesstoken"
-            | "signature"
-            | "secret"
-            | "password"
-            | "token"
-            | "clientsecret"
-            | "awssecretaccesskey"
-            | "awsaccesskeyid"
-            | "awssessiontoken"
+        "cookie" | "setcookie" | "key" | "awsaccesskeyid"
     ) || [
         "apikey",
         "secret",