diff --git a/litellm-rust/crates/secrets-cyberark/src/secret_manager.rs b/litellm-rust/crates/secrets-cyberark/src/secret_manager.rs index ade257a5872..ee4bded4b29 100644 --- a/litellm-rust/crates/secrets-cyberark/src/secret_manager.rs +++ b/litellm-rust/crates/secrets-cyberark/src/secret_manager.rs @@ -116,13 +116,9 @@ impl CyberArkSecretManager { .get(CYBERARK_REFRESH_INTERVAL) .map(|value| { value - .parse::() + .parse::() + .map(Duration::from_secs) .map_err(|_| Error::RefreshInterval) - .map(|seconds| match seconds { - seconds if seconds < 0 => Duration::from_nanos(1), - 0 => DEFAULT_REFRESH_INTERVAL, - seconds => Duration::from_secs(seconds as u64), - }) }) .transpose()?; Ok(Self::with_client( diff --git a/litellm-rust/crates/secrets-cyberark/tests/secret_manager.rs b/litellm-rust/crates/secrets-cyberark/tests/secret_manager.rs index e8158c46945..9ea8da63383 100644 --- a/litellm-rust/crates/secrets-cyberark/tests/secret_manager.rs +++ b/litellm-rust/crates/secrets-cyberark/tests/secret_manager.rs @@ -2,7 +2,7 @@ use std::{sync::Arc, time::Duration}; use base64::{Engine, engine::general_purpose::STANDARD}; use litellm_secrets_cyberark::{CyberArkSecretManager, DeleteOutcome, Error}; -use litellm_secrets_types::{SecretValue, validate_secret_name}; +use litellm_secrets_types::SecretValue; use serde::Deserialize; use wiremock::{ Match, Mock, MockServer, Request, ResponseTemplate, @@ -428,11 +428,6 @@ fn new_reports_missing_client_certificate_files() { )); } -#[test] -fn secret_name_validation_matches_write_guard() { - assert!(validate_secret_name("../etc").is_err()); -} - #[tokio::test] async fn trailing_slash_endpoint_preserves_base_path() { let server = MockServer::start().await;