Skip to content

Commit d9d51e9

Browse files
committed
docs(gateway): drop the stale 'optional' wording on cert_validator
cert_validator is an Arc<dyn CertValidator>, not an Option, but both its own doc comment and HttpsClientConfig's still described it as optional.
1 parent c879580 commit d9d51e9

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

dstack/gateway/src/kv/https_client.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,14 +53,14 @@ pub trait CertValidator: Debug + Send + Sync + 'static {
5353
fn validate(&self, cert_der: &[u8]) -> Result<(), String>;
5454
}
5555

56-
/// TLS configuration for mTLS with optional custom certificate validation
56+
/// TLS configuration for mTLS with custom certificate validation
5757
#[derive(Clone)]
5858
pub struct HttpsClientConfig {
5959
pub cert_path: String,
6060
pub key_path: String,
6161
pub ca_cert_path: String,
62-
/// Optional custom certificate validator (checked during TLS handshake)
63-
/// Checked against the peer's certificate during the handshake.
62+
/// Custom certificate validator, checked against the peer's certificate
63+
/// during the handshake.
6464
///
6565
/// Not optional: this is the only thing that distinguishes a gateway in our own
6666
/// cluster from any other holder of a certificate the shared CA signed, and CA-path

0 commit comments

Comments
 (0)