From 0ac8178baa1a07f296ef5594b63a1e8ea83a1e5f Mon Sep 17 00:00:00 2001
From: "Rodrigo Rodriguez (Pragmatismo)" <me@rodrigorodriguez.com>
Date: Sun, 5 Apr 2026 01:19:45 -0300
Subject: [PATCH] fix: use std::sync::RwLock for cache to support sync methods

---
 src/core/secrets/mod.rs | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/core/secrets/mod.rs b/src/core/secrets/mod.rs
index 2eac0bbd..39abedf2 100644
--- a/src/core/secrets/mod.rs
+++ b/src/core/secrets/mod.rs
@@ -8,6 +8,7 @@ use std::path::PathBuf;
 use std::sync::Arc;
 use std::sync::Arc as StdArc;
 use tokio::sync::RwLock;
+use std::sync::RwLock as StdRwLock;
 use uuid::Uuid;
 use vaultrs::client::{VaultClient, VaultClientSettingsBuilder};
 use vaultrs::kv2;
@@ -62,7 +63,7 @@ struct CachedSecret {
 #[derive(Clone)]
 pub struct SecretsManager {
     client: Option<StdArc<VaultClient>>,
-    cache: Arc<RwLock<HashMap<String, CachedSecret>>>,
+    cache: Arc<std::sync::RwLock<HashMap<String, CachedSecret>>>,
     cache_ttl: u64,
     enabled: bool,
 }
@@ -106,7 +107,7 @@ impl SecretsManager {
             warn!("Vault not configured. Using environment variables directly.");
             return Ok(Self {
                 client: None,
-                cache: Arc::new(RwLock::new(HashMap::new())),
+                cache: Arc::new(StdRwLock::new(HashMap::new())),
                 cache_ttl,
                 enabled: false,
             });