Skip to content

Commit c3878d8

Browse files
fix: Do not allow tls connections to Vault with TLS Verification
Fixes NVBug 5999702 Signed-off-by: Ian Anderson <ianderson@nvidia.com>
1 parent d4dfa0d commit c3878d8

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

crates/secrets/src/forge_vault.rs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,11 @@ where
106106
.ca_certs(vec![vault_client_config.vault_root_ca_path.clone()])
107107
.verify(true)
108108
} else {
109-
vault_client_settings_builder.verify(false)
109+
tracing::error!(
110+
"Vault root CA not found at {}. Refusing to connect without TLS verification.",
111+
vault_client_config.vault_root_ca_path
112+
);
113+
return Err(eyre!("Vault root CA not found"));
110114
};
111115

112116
Ok(vault_client_settings_builder.build()?)

0 commit comments

Comments
 (0)